-
Notifications
You must be signed in to change notification settings - Fork 601
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
Restructure and update Jakarta authorization code #30293
base: integration
Are you sure you want to change the base?
Conversation
!build |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=3fa5602b-7025-4fa2-9cfd-32a79f1cdca6 Target locations of links might be accessible only to IBM employees. |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=525a905a-882a-484f-a861-27d8b73e532f Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
!build spawn.fullfat.buckets=com.ibm.ws.ejbcontainer.security.jacc_fat,com.ibm.ws.ejbcontainer.security.jacc_fat.2,com.ibm.ws.ejbcontainer.security.jacc_fat.2.1,com.ibm.ws.webcontainer.security.jacc.1.5_fat,com.ibm.ws.security.jaspic_fat |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=bc6dbca4-a0f0-4cbf-b3a6-f40cc49c29c2 Target locations of links might be accessible only to IBM employees. |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=d0a0c5a9-5a09-4c73-b66b-374a077a8285 Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
1 similar comment
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
!build spawn.fullfat.buckets=com.ibm.ws.ejbcontainer.security.jacc_fat,com.ibm.ws.ejbcontainer.security.jacc_fat.2,com.ibm.ws.ejbcontainer.security.jacc_fat.2.1,com.ibm.ws.webcontainer.security.jacc.1.5_fat,com.ibm.ws.security.jaspic_fat |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=cff28fa3-a0ae-48dc-9c67-35e2638a1ee7 Target locations of links might be accessible only to IBM employees. |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=82b17b9b-b517-4b8a-9a5a-fbd9473c67da Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
1 similar comment
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
!build spawn.fullfat.buckets=com.ibm.ws.ejbcontainer.security.jacc_fat,com.ibm.ws.ejbcontainer.security.jacc_fat.2,com.ibm.ws.ejbcontainer.security.jacc_fat.2.1,com.ibm.ws.webcontainer.security.jacc.1.5_fat,com.ibm.ws.security.jaspic_fat |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=0596815a-bbc0-4504-880d-eca277eca3e0 Target locations of links might be accessible only to IBM employees. |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=d5949def-2f8d-4b92-a8d2-2ba0065eff31 Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
1 similar comment
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
!build spawn.fullfat.buckets=com.ibm.ws.ejbcontainer.security.jacc_fat,com.ibm.ws.ejbcontainer.security.jacc_fat.2,com.ibm.ws.ejbcontainer.security.jacc_fat.2.1,com.ibm.ws.webcontainer.security.jacc.1.5_fat,com.ibm.ws.security.jaspic_fat |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=f81a7533-062d-4d22-93a9-8632ec7d5c3a Target locations of links might be accessible only to IBM employees. |
Your personal pipeline request is at https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=3f6f3a1f-2eda-4f38-b516-0b5e231ac928 Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
1 similar comment
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=f81a7533-062d-4d22-93a9-8632ec7d5c3a - Open Liberty Personal Build RTC completed with errors/failures. |
!libby@dev (view Open Liberty Delivery Requirements Verification - ❌ completed with errors/failures.) |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
@jhanders34 please bear with me while I run test pipelines to fix the bug we've been discussing in the past few weeks. |
Please ignore! !libby (view Open Liberty Delivery Requirements Verification - ✅ completed successfully!) |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
- Update to have interfaces in EJB and Web bundles so that can reference actual classes instead of using Object - Move JaccService function to the corresponding EJB and Web projects to keep the methods with their appropriate container function - Update to add PrincipalMapper to PolicyContextHandlerImpl in preparation of adding PrincipalMapper implementation - Update to use instance methods instead of static methods in PolicyConfigurationManager - Inline most of the Validator and Propagator function into the EJB and Web classes except for EJB propagator which is used by both EJB function and base Jacc function
!build (view Open Liberty Personal Build - ❌ completed with errors/failures.) Note: Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
Sorry, Jared, please ignore any resulting pipelines. This is for testing bug we discussed. !libby@dev Note: Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
!build Note: Target locations of links might be accessible only to IBM employees. |
Code analysis and actionsDO NOT DELETE THIS COMMENT.
|
Update to have interfaces in EJB and Web bundles so that can reference actual classes instead of using Object
Move JaccService function to the corresponding EJB and Web projects to keep the methods with their appropriate container function
Update to add PrincipalMapper to PolicyContextHandlerImpl in preparation of adding PrincipalMapper implementation
Update to use instance methods instead of static methods in PolicyConfigurationManager
Inline most of the Validator and Propagator function into the EJB and Web classes except for EJB propagator which is used by both EJB function and base Jacc function
I have considered the risk of behavior change or other zero migration impact (https://github.com/OpenLiberty/open-liberty/wiki/Behavior-Changes).
If this PR fixes an Issue, the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN" (verify
release bug
label if applicable: https://github.com/OpenLiberty/open-liberty/wiki/Open-Liberty-Conventions).If this PR resolves an external Known Issue (including APARS), the description includes "Fixes #FILLMEIN" or "Resolves #FILLMEIN".