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

[Spring Cleanup] Refactor oauth2 webapp to remove Spring Framework Dependency #2675

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

lashinijay
Copy link
Contributor

@lashinijay lashinijay commented Jan 16, 2025

Proposed changes in this pull request

  • $subject

Related Issue(s)

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12812502295

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12812502295
Status: cancelled

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12821617859

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12821617859
Status: cancelled

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12822396715

nilasini
nilasini previously approved these changes Jan 17, 2025
Copy link

codecov bot commented Jan 17, 2025

Codecov Report

Attention: Patch coverage is 48.57143% with 18 lines in your changes missing coverage. Please review.

Project coverage is 56.17%. Comparing base (32cb723) to head (15433b9).

Files with missing lines Patch % Lines
...bon/identity/oauth/endpoint/util/EndpointUtil.java 37.50% 9 Missing and 1 partial ⚠️
...tity/oauth/endpoint/authz/OAuth2AuthzEndpoint.java 20.00% 7 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2675      +/-   ##
============================================
+ Coverage     55.66%   56.17%   +0.50%     
+ Complexity     8435     8189     -246     
============================================
  Files           632      631       -1     
  Lines         48587    46711    -1876     
  Branches       9300     8851     -449     
============================================
- Hits          27048    26240     -808     
+ Misses        17670    16710     -960     
+ Partials       3869     3761     -108     
Flag Coverage Δ
unit 38.84% <48.57%> (-0.14%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12822396715
Status: failure

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12824946667

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12824946667
Status: failure

@jenkins-is-staging
Copy link

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12825653251

@jenkins-is-staging
Copy link

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12825653251
Status: failure

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.

3 participants