Skip to content

Commit

Permalink
Change default identityProviderType to INTERNAL_IDP in stroom pro…
Browse files Browse the repository at this point in the history
…d config
  • Loading branch information
at055612 committed Jan 9, 2025
1 parent 9d57ad5 commit 03929b9
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 4 deletions.
8 changes: 4 additions & 4 deletions stroom-app/prod.yml.jinja2
Original file line number Diff line number Diff line change
Expand Up @@ -205,11 +205,11 @@ appConfig:
security:
authentication:
openId:
# (EXTERNAL_IDP|NO_IDP|TEST_CREDENTIALS), Only use TEST_CREDENTIALS for test/demo installs
# NO_IDP - Use when stroom is using its internal IDP. Use apiKey for feedStatus checks.
# Use certificates for forwarding.
# (EXTERNAL_IDP|INTERNAL_IDP|TEST_CREDENTIALS), Only use TEST_CREDENTIALS for test/demo installs
# INTERNAL_IDP - Use API Keys generated in Stroom for feed status check. No support for token
# authentication on data receipt.
# EXTERNAL_IDP - Usa an external IDP for receipt token auth and for outbound HTTP connections
identityProviderType: "{{ 'NO_IDP' | envVar('IDENTITY_PROVIDER_TYPE') }}"
identityProviderType: "{{ 'INTERNAL_IDP' | envVar('IDENTITY_PROVIDER_TYPE') }}"
{% endif %}
serviceDiscovery:
enabled: "{{ 'false' | envVar('STROOM_SERVICE_DISCOVERY_ENABLED') }}"
Expand Down
19 changes: 19 additions & 0 deletions unreleased_changes/20250109_100540_477__0.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
* Change default `identityProviderType` to `INTERNAL_IDP` in stroom prod config.


```sh
# ONLY the top line will be included as a change entry in the CHANGELOG.
# The entry should be in GitHub flavour markdown and should be written on a SINGLE
# line with no hard breaks. You can have multiple change files for a single GitHub issue.
# The entry should be written in the imperative mood, i.e. 'Fix nasty bug' rather than
# 'Fixed nasty bug'.
#
# Examples of acceptable entries are:
#
#
# * Issue **123** : Fix bug with an associated GitHub issue in this repository
#
# * Issue **namespace/other-repo#456** : Fix bug with an associated GitHub issue in another repository
#
# * Fix bug with no associated GitHub issue.
```

0 comments on commit 03929b9

Please sign in to comment.