Skip to content

/ simplesamlphp

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.

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

Remove the SimpleSAML_Auth_Default class #151

Closed
jaimeperez opened this issue Jan 15, 2015 · 2 comments
Closed

Remove the SimpleSAML_Auth_Default class #151

jaimeperez opened this issue Jan 15, 2015 · 2 comments
Labels
enhancement maintainability started
Milestone
2.0

Comments

@jaimeperez
Copy link
Member

@jaimeperez jaimeperez commented Jan 15, 2015

This one is tricky, as there's lots of code currently in use. Try to reallocate those parts into SimpleSAML_Auth_Source and SimpleSAML_Session.
@jaimeperez jaimeperez added this to the v2.0 milestone Jan 15, 2015
@jaimeperez
Copy link
Member Author

@jaimeperez jaimeperez commented Sep 1, 2015

The class has been deprecated now, all the code in use reallocated, and nothing points to it any longer.
@jaimeperez jaimeperez added the started label Sep 1, 2015
@tvdijen
Copy link
Member

@tvdijen tvdijen commented Dec 11, 2018

We should be able to safely remove this after 3yrs of deprecation
@tvdijen tvdijen closed this in 2c8aa8a Feb 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement maintainability started
Projects
None yet
Milestone
2.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@tvdijen @jaimeperez