Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Release 10.0.0-alpha.1 - included modules #201

Merged
merged 110 commits into from
May 30, 2024
Merged

Release 10.0.0-alpha.1 - included modules #201

merged 110 commits into from
May 30, 2024

Conversation

briskt
Copy link
Contributor

@briskt briskt commented May 27, 2024 •
edited
Loading

Added

  • Included SIL modules directly in this repository: expirychecker, material, mfa, profilereview, silauth, and sildisco.
  • Include a default authsources.php file suitable for use in an IdP instance.

Changed (breaking)

  • Changed the namespace on the Validator class to SimpleSAML\Module\expirychecker.
  • Changed the namespace on the Assert and LoggerFactory classes to SimpleSAML\Module\profilereview.
  • Changed the namespace on the Assert, LoggerFactory, and LoginBrowser classes to SimpleSAML\Module\mfa.
  • Changed the namespace on the silauth module source files to SimpleSAML\Module\silauth. This is particularly important to note if the ConfigManager class is used in authsources.php or other files copied into the ssp-base image at deployment time.

@briskt
Merge branch 'main' into develop
ca08002
@briskt
copy content of the expirychecker module
b3d3f3d 
https://github.com/silinternational/simplesamlphp-module-expirychecker
@briskt
use the local expirychecker code and remove from composer
d1bfa72
@briskt
move Validator into lib directory to avoid need for custom autoload
bffa321
@briskt
add behat tests from the expirychecker repo
93fcfbe
@briskt
remove redundant test scenarios
bc493f0
@briskt
remove behat.yml from Docker image and add it to actions-services.yml
7d63fae
@briskt
copy content of the profilereview module
9057509 
https://github.com/silinternational/simplesamlphp-module-profilereview
@briskt
use the local profilereview code and remove from composer
d9c0772
@briskt
move Assert and LoggerFactory into lib directory and change namespace
d446c84
@briskt
add behat tests from the profilereview repo
6ecd48b
@briskt
add profilereview users into authsources.php
be69053
@briskt
fix profilereview tests to run correctly here
ccb25b0 
except the ones that look for specific strings fail because of the material theme
@briskt
enable test IDP2; use "default" theme on IDP1 and "material" on IDP2
81d06fd
@briskt
update actions-services.yml with test IDP changes
c8908fd
@briskt
add ssp-idp2.local to depends_on
ad48854
@briskt
change expirychecker namespace to ExpiryChecker for aesthetics
1e694e9
@briskt
Revert "change expirychecker namespace to ExpiryChecker for aesthetics"
d1f328d 
This reverts commit 1e694e9.
@briskt
Merge pull request #193 from silinternational/feature/expirychecker
17a9828 
directly include expirychecker module
@briskt
Merge branch 'develop' into feature/profilereview
b463f4d
@briskt
use a different password for each profilereview user
4b73362
@briskt
add module variables to local.env.dist file [skip ci]
04c34b1
@briskt
copy content from the mfa module repo
09380a6 
https://github.com/silinternational/simplesamlphp-module-mfa
@briskt
use the local mfa module code and remove external module from composer
34f1256
@briskt
composer require silinternational/idp-id-broker-php-client
8262e5f
@briskt
composer require sinergi/browser-detector
294ff16
@briskt
move Assert, LoggerFactory, LoginBrowser to lib directory
b87c919
@briskt
set up MfaContext for behat
5dbc2c3
@briskt
Merge pull request #194 from silinternational/feature/profilereview
220f3f0 
directly include the profilereview module
@briskt
fixed (some of) the mfa tests
de593ea
@briskt briskt changed the title Release 9.4.0 - included modules Release 10.0.0 - included modules May 28, 2024
@briskt briskt changed the title Release 10.0.0 - included modules Release 10.0.0-alpha.1 - included modules May 28, 2024
briskt added 12 commits May 29, 2024 16:26
@briskt
Google analytics4
44baf14 
silinternational/simplesamlphp-module-material#119
@briskt
input color was too light
9d1d029 
silinternational/simplesamlphp-module-material#136
@briskt
PHP 8 update
3f9a018 
silinternational/simplesamlphp-module-material#108
@briskt
fixed bug with float type sent to translate, added manual testing scen
5671793 
silinternational/simplesamlphp-module-material#113
@briskt
add logo caption
6ccbb26 
silinternational/simplesamlphp-module-material#115
silinternational/simplesamlphp-module-material#121
silinternational/simplesamlphp-module-material#123
@briskt
IDP-103 Have the browser require the username and password on login page
3ae33b2 
silinternational/simplesamlphp-module-material#125
@briskt
Add width related styling to the header in order to allow the word wr…
cee96f3 
…ap to styling to function

https://github.com/silinternational/simplesamlphp-module-material/pull/127/files
@briskt
Tell user which IdP the smartphone app is needed for
e147782 
silinternational/simplesamlphp-module-material#129
@briskt
idp name reminder on TOTP form
31e3cfc 
https://github.com/silinternational/simplesamlphp-module-material/pull/131/files
@briskt
Improve English "Send code to manager" wording
92f26dd 
https://github.com/silinternational/simplesamlphp-module-material/pull/107/files
@briskt
replace smartphone with authenticator IDP-24
5301b69 
silinternational/simplesamlphp-module-material#134
@briskt
Merge pull request #210 from silinternational/feature/catch-up
291e21e 
Catch up on recent changes made to material module repo
@mtompset
Copy link

Sorry, this is too painfully long to consider reviewing tonight.

@briskt
Copy link
Contributor Author

briskt commented May 30, 2024

Sorry, this is too painfully long to consider reviewing tonight.

I might just bypass restrictions and merge it without review to save everyone the anxiety.

@briskt
Copy link
Contributor Author

briskt commented May 30, 2024

Thank you, @jason-jackson !

@briskt briskt merged commit a953fea into main May 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jason-jackson jason-jackson jason-jackson approved these changes

@forevermatt forevermatt Awaiting requested review from forevermatt

@devon-sil devon-sil Awaiting requested review from devon-sil

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@briskt @mtompset @jason-jackson