So far we have explicitly listed all OAUTH clients, with dedicated
configuration variables. This does not work when integrating external
components, so as a more flexible meachnism we pick up on
auth.extra_passport_clients, and allow injecting that value via the
config/override.php mechanism (which is the same as used in
AppServiceProvider.php).
Details
Details
- Reviewers
machniak - Group Reviewers
Restricted Project - Commits
- rK86affc744294: Support injecting extra passport clients via override.php
Diff Detail
Diff Detail
- Repository
- rK kolab
- Branch
- dev/mollekopf
- Lint
Lint Skipped - Unit
No Test Coverage - Build Status
Buildable 55287 Build 19653: arc lint + arc unit
Event Timeline
Comment Actions
Alternative approaches I've considered:
- Somehow inject the configuration into auth.php: Seems messy because we'd have to modify the file when starting the container (the base version is part of the container)
- Use the regular "env variable picked up by the configuration" approach: The data seems just too complex to stuff through an env variable (but maybe that's not actually true if it is just a blob of json?).
- Use a custom init command via overlay: Possible, but also more complicated it would seem.