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

Move OAuth2 Core Server Classes to UAA namespace + Tests #2848

Open
wants to merge 133 commits into
base: develop
Choose a base branch
from

Conversation

because of parallel tests
- Endpoints for OAuth2 - main OIDC endpoints are in UAA implemented
- Core-Beans for Server start
- Client Token
- Server Annotation forking (workaround for now)

- OPEN. Client Annotation
Includes Test-Framework for OAuth2
Includes Test-Framework for OAuth2
- remove csrf disable because we dont use this init and codeql dont like it
Remove AuthorizationEndpoint.java because UAA has its own implementation
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/187485301

The labels on this github issue will be updated when the story is started.

Changed interfaces without wildcards
The interfaces now belong to UAA
Uaa has forked this class in the past, re-use it now
…Factory

Use IdentityZoneManager
Remove internal OAuth2RequestFactory proxy
…auth2/core-server

# Conflicts:
#	server/src/test/java/org/cloudfoundry/identity/uaa/oauth/ClientAccessTokenValidityTest.java
#	server/src/test/java/org/cloudfoundry/identity/uaa/oauth/ClientRefreshTokenValidityTest.java
#	server/src/test/java/org/cloudfoundry/identity/uaa/oauth/TokenRevocationEndpointTests.java
#	server/src/test/java/org/cloudfoundry/identity/uaa/oauth/expression/IsSelfCheckTest.java
#	server/src/test/java/org/cloudfoundry/identity/uaa/zone/MultitenantJdbcClientDetailsServiceTests.java
#	uaa/src/test/java/org/cloudfoundry/identity/uaa/login/TokenEndpointDocs.java
#	uaa/src/test/java/org/cloudfoundry/identity/uaa/mock/token/TokenMvcMockTests.java
@strehle strehle marked this pull request as ready for review May 2, 2024 10:39
@strehle strehle requested a review from a team May 2, 2024 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants