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.

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

[READY] 2.x - Deprecate idp_cert_fingerprint / idp_cert_fingerprint_algorithm + README cleanup #701

Merged
merged 5 commits into from
Jan 19, 2025

Conversation

johnnyshields
Copy link
Collaborator

@johnnyshields johnnyshields commented Jul 9, 2024

  • Deprecate idp_cert_fingerprint and idp_cert_fingerprint_algorithm options (to remove in 2,1.0)
  • Improvements to README:
    • Edits for clarity & brevity
    • Replace terminology "authorization" with "authentication"
    • Remove OneLogin references
    • Spelling fixes, etc.
    • Replace hash rockets with colons

- Edits for clarity & brevity
- Replace terminology "authorization" with "authentication"
- Remove OneLogin references
- Spelling fixes, etc.
- Replace hash rockets with colons
@johnnyshields
Copy link
Collaborator Author

@pitbulk this is ready for your final review. Please fix/revert anything you find yourself, no need to run it by me.

After merging this, let's release 2.0.0 🎉 I'm already running the latest 2.x branch in production without issues.

@johnnyshields johnnyshields changed the title [DO NOT MERGE] 2.x README cleanup [READY] 2.x README cleanup Jan 19, 2025
@johnnyshields johnnyshields changed the title [READY] 2.x README cleanup [READY] 2.x - Deprecate idp_cert_fingerprint / idp_cert_fingerprint_algorithm + README cleanup Jan 19, 2025
@pitbulk pitbulk merged commit 8b21a84 into SAML-Toolkits:v2.x Jan 19, 2025
20 of 21 checks passed
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.

2 participants