Skip to content

feat: store authsources in a database (fix #267)#1817

Open
damikael wants to merge 323 commits intosimplesamlphp:masterfrom
linfaservice:master
Open

feat: store authsources in a database (fix #267)#1817
damikael wants to merge 323 commits intosimplesamlphp:masterfrom
linfaservice:master

Conversation

@damikael
Copy link
Copy Markdown
Contributor

If, into config/config.php, are present the parameters:

'authsources.storage' => 'database',
'authsources.database_table' => 'authsources',

then authsources are retrieved from database,
else them are retrieved from file same authsources.php, as before.

If the value of parameter authsources.storage is 'database', then the authsource configuration is retrieved from database.
If the table referred by the parameter authsources.database_table does not exists it will be created.
The table will contain the following fields:

  • id VARCHAR(255) PRIMARY KEY, that is the key of authsource
  • entity_data JSON, that is the configuration of authsource
  • _disabled enum('N','Y') NOT NULL DEFAULT 'N', to enable/disable the authsource

tvdijen and others added 30 commits September 22, 2022 08:45
Bumps [twig/twig](https://github.com/twigphp/Twig) from 3.4.2 to 3.4.3.
- [Release notes](https://github.com/twigphp/Twig/releases)
- [Changelog](https://github.com/twigphp/Twig/blob/3.x/CHANGELOG)
- [Commits](twigphp/Twig@v3.4.2...v3.4.3)

---
updated-dependencies:
- dependency-name: twig/twig
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* entityID change affects saml:SP too

* Generate exception for example entityID (SP)

* Make entityId consistent with examples

* Generate exception for example entityID (IdP)

* Add some use-statements

Co-authored-by: Tim van Dijen <tvdijen@gmail.com>
Storing this in each parsed entity has a significant performance impact.
It was originally added in fe80b8a for
use by the 'aggregator' module, which is now replaced by the 'aggregator2'
module which does not use this field.
Forward port of @tauceti2 patch in simplesamlphp#833
to apply cleanly against simplesamlphp-2.0.0-rc2
Bumps [loader-utils](https://github.com/webpack/loader-utils) from 2.0.2 to 2.0.3.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v2.0.3/CHANGELOG.md)
- [Commits](webpack/loader-utils@v2.0.2...v2.0.3)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Change build-script to leverage composer archive

* No longer include data/ cache/ and log/ directories. They don't belong in the install-dir

* Remove unnecessary .gitkeep files

* Merge config&config-templates and metadata&metadata-templates; templates renamed to .dist

* Reduce git clone overhead

* Throw an exception if datadir not set

Co-authored-by: Thijs Kinkhorst <thijs@kinkhorst.com>
Bumps [loader-utils](https://github.com/webpack/loader-utils) from 2.0.3 to 2.0.4.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v2.0.4/CHANGELOG.md)
- [Commits](webpack/loader-utils@v2.0.3...v2.0.4)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 7587851 to d523b31 Compare August 2, 2023 11:58
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from 8c90121 to d534e3b Compare September 5, 2023 08:09
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from bc1c5c8 to d0a5974 Compare October 17, 2023 21:17
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from ccb9b02 to 120a100 Compare December 1, 2023 14:34
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 6004a77 to 58bf8db Compare May 4, 2024 23:45
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 5c9fb2c to 0970efc Compare May 27, 2024 12:27
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from c27831c to 71e49f4 Compare June 19, 2024 17:03
@tvdijen tvdijen force-pushed the master branch 3 times, most recently from c06a17a to a52c98d Compare August 20, 2025 21:45
@tvdijen tvdijen force-pushed the master branch 2 times, most recently from 76da272 to f10b701 Compare March 18, 2026 21:37
@tvdijen tvdijen force-pushed the master branch 7 times, most recently from 5f06278 to 4dd709e Compare March 23, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.