build(deps): bump opensaml2 from 2.6.6 to 4.0.1 #2908
+23,751
−17,364
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tallicia
added
in progress
DO NOT MERGE
|
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/187710947 The labels on this github issue will be updated when the story is started. |
Tallicia
changed the title
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
...ava/org/cloudfoundry/identity/uaa/provider/saml/SamlLegacyAliasResponseForwardingFilter.java
Fixed
Show fixed
Hide fixed
duanemay
force-pushed
the
new-saml-0530
branch
5 times, most recently
from
65b0d64
to
e67a40a
Compare
duanemay
force-pushed
the
new-saml-0530
branch
9 times, most recently
from
a761b67
to
7de27a1
Compare
duanemay
force-pushed
the
new-saml-0530
branch
2 times, most recently
from
0d3a595
to
f199f50
Compare
peterhaochen47
force-pushed
the
new-saml-0530
branch
from
46248b9
to
290f89c
Compare
duanemay
force-pushed
the
new-saml-0530
branch
2 times, most recently
from
46248b9
to
b6cb65b
Compare
peterhaochen47
force-pushed
the
new-saml-0530
branch
2 times, most recently
from
a97457f
to
745fff3
Compare
strehle
changed the title
* Cleanup libraries not needed anymore bound to old opensaml * Remove ESAPI finally this dependency is only there because of old saml * fix rebase
# Conflicts: # dependencies.gradle
Will add more coverage in Saml2Utils
Signed-off-by: Duane May <[email protected]>
strehle
reviewed
Nov 14, 2024
.../cloudfoundry/identity/uaa/provider/saml/ConfiguratorRelyingPartyRegistrationRepository.java
Outdated
Show resolved
Hide resolved
.../cloudfoundry/identity/uaa/provider/saml/ConfiguratorRelyingPartyRegistrationRepository.java
Outdated
Show resolved
Hide resolved
...ava/org/cloudfoundry/identity/uaa/provider/saml/Saml2BearerGrantAuthenticationConverter.java
Outdated
Show resolved
Hide resolved
...ava/org/cloudfoundry/identity/uaa/provider/saml/Saml2BearerGrantAuthenticationConverter.java
Outdated
Show resolved
Hide resolved
...ava/org/cloudfoundry/identity/uaa/provider/saml/Saml2BearerGrantAuthenticationConverter.java
Show resolved
Hide resolved
server/src/main/java/org/cloudfoundry/identity/uaa/provider/saml/Saml2Utils.java
Show resolved
Hide resolved
|
SAML2 bearer and IdP initated flows are similar and the IdP initiated flow still is not working. The resolver needs to be forked / implemented on Uaa side, I assume then we can resolve from issuer to idP . SAML2 bearer works now independent from amount of SAML2 trusts, #3132 |
* Test saml bearer * Fixes for SAML2 bearer flow * reverted test
# Conflicts: # dependencies.gradle
* Test saml bearer * Fixes for SAML2 bearer flow * reverted test * Implement RelyingPartyRegistrationResolver * support resolution of SAMLResponse from request * remove default setting * Use standard setting of metadata the feature with classpath is new in this PR. * refactorings based on sonar * Replace dummy-saml-idp-metadata and create the data based on real key data Until now we do not deliver any keys in uaa.war. * Cleanup test failure Changed, because of hack with defaults. * Rename DefaultRelyingPartyRegistrationResolver to UaaRelyingPartyRegistrationResolver Signed-off-by: Duane May <[email protected]> * Refactor text blocks Signed-off-by: Duane May <[email protected]> --------- Signed-off-by: Duane May <[email protected]> Co-authored-by: Duane May <[email protected]> Co-authored-by: Duane May <[email protected]>
# Conflicts: # dependencies.gradle
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Replacing the other feature branch #2862 for new SAML library replacement effort.
Sonar
https://sonarcloud.io/summary/new_code?id=cloudfoundry-identity-parent&pullRequest=2908