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

Allow OAuth2 application management through DCR in sub organizations #2578

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ShanChathusanda93
Copy link
Contributor

@ShanChathusanda93 ShanChathusanda93 commented Sep 30, 2024

Proposed changes in this pull request

  • $subject
  • Improvement for : Support for creating OAuth 2.0 clients inside B2B organizations for B2B API access wso2/product-is#21208
  • OAuth2 application management will be allowed through the DCR endpoint.
    /t/{tenant-domain}/o/{org-id}/api/identity/oauth2/dcr/v1.1/register
  • This change contains the following improvements in the sub organization level
    • OAuth2 application creation
    • OAuth2 application get by client id
    • OAuth2 application get by client / application name
    • OAuth2 application update by client id
    • OAuth2 application delete by client id
  • Contains some supportive method improvements to the aforementioned main operations.

When should this PR be merged

@ShanChathusanda93 ShanChathusanda93 force-pushed the dcr-impr-branch branch 3 times, most recently from 92e8d33 to 725f30e Compare November 1, 2024 13:16
pom.xml Outdated
@@ -933,7 +933,7 @@
<apache.felix.scr.ds.annotations.version>1.2.4</apache.felix.scr.ds.annotations.version>

<!-- Carbon kernel version -->
<carbon.kernel.version>4.10.22</carbon.kernel.version>
<carbon.kernel.version>4.10.24</carbon.kernel.version>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to update the kernel version to the new released version with the changes in wso2/carbon-kernel#4104

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to the latest kernel release 4.10.26[1]

[1] https://github.com/wso2/carbon-kernel/tree/v4.10.26

@ShanChathusanda93 ShanChathusanda93 changed the title Improve DCR process in OAuth Allow OAuth2 application creation from DCR in sub organizations Nov 1, 2024
@ShanChathusanda93 ShanChathusanda93 changed the title Allow OAuth2 application creation from DCR in sub organizations Allow OAuth2 application management through DCR in sub organizations Nov 1, 2024
@ShanChathusanda93 ShanChathusanda93 marked this pull request as ready for review November 1, 2024 13:27
@ShanChathusanda93 ShanChathusanda93 force-pushed the dcr-impr-branch branch 2 times, most recently from f7ec0ad to dd68cd0 Compare November 27, 2024 08:19
@ShanChathusanda93 ShanChathusanda93 force-pushed the dcr-impr-branch branch 2 times, most recently from ea2d920 to 8ddbb51 Compare November 28, 2024 05:57
@Thumimku
Copy link
Contributor

Hi @ShanChathusanda93,

⚠️ Notice: This PR has been open for a while. To keep the repository clean and up-to-date, this PR will be closed within the next two weeks if there is no further activity.

Please take one of the following actions:

  • Merge the PR if it is ready.
  • Close the PR if it is no longer relevant.
  • Leave a comment explaining why it should remain open and provide an update on its progress.

Your prompt attention to this matter is greatly appreciated. Thank you for your understanding and collaboration! 🙏

@ShanChathusanda93
Copy link
Contributor Author

Hi @ShanChathusanda93,

⚠️ Notice: This PR has been open for a while. To keep the repository clean and up-to-date, this PR will be closed within the next two weeks if there is no further activity.

Please take one of the following actions:

  • Merge the PR if it is ready.
  • Close the PR if it is no longer relevant.
  • Leave a comment explaining why it should remain open and provide an update on its progress.

Your prompt attention to this matter is greatly appreciated. Thank you for your understanding and collaboration! 🙏

@Thumimku

This PR is in WIP and will be completed in next couple of days. Hence keeping this open as for now.

@ShanChathusanda93 ShanChathusanda93 force-pushed the dcr-impr-branch branch 3 times, most recently from a670f83 to 5637863 Compare December 2, 2024 12:26
Copy link

codecov bot commented Dec 2, 2024

Codecov Report

Attention: Patch coverage is 42.98246% with 65 lines in your changes missing coverage. Please review.

Project coverage is 56.39%. Comparing base (654bc26) to head (a6843bf).

Files with missing lines Patch % Lines
...rg/wso2/carbon/identity/oauth/dao/OAuthAppDAO.java 12.00% 20 Missing and 2 partials ⚠️
...g/wso2/carbon/identity/oauth2/util/OAuth2Util.java 45.71% 15 Missing and 4 partials ⚠️
...entity/oauth/dcr/internal/DCRServiceComponent.java 0.00% 6 Missing ⚠️
...carbon/identity/oauth/dcr/service/DCRMService.java 78.57% 5 Missing and 1 partial ⚠️
...2/carbon/identity/oauth/OAuthAdminServiceImpl.java 50.00% 5 Missing and 1 partial ⚠️
...bon/identity/oauth/dcr/internal/DCRDataHolder.java 0.00% 3 Missing ⚠️
.../wso2/carbon/identity/oauth/OAuthAdminService.java 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2578      +/-   ##
============================================
- Coverage     56.43%   56.39%   -0.04%     
+ Complexity     8188     8186       -2     
============================================
  Files           632      632              
  Lines         46323    46410      +87     
  Branches       8245     8253       +8     
============================================
+ Hits          26141    26172      +31     
- Misses        16469    16514      +45     
- Partials       3713     3724      +11     
Flag Coverage Δ
unit 38.66% <42.98%> (+<0.01%) ⬆️

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.

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.

4 participants