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

Restructure and update Jakarta authorization code #30293

Open
wants to merge 2 commits into
base: integration
Choose a base branch
from

Conversation

jhanders34
Copy link
Member

@jhanders34 jhanders34 commented Nov 25, 2024

  • 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".

@jhanders34 jhanders34 self-assigned this Nov 25, 2024
@jhanders34
Copy link
Member Author

!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

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

  • 1 infrastructure code files were changed.

@LibbyBot
Copy link

@LibbyBot
Copy link

@jhanders34
Copy link
Member Author

!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

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@jhanders34
Copy link
Member Author

!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

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@jhanders34
Copy link
Member Author

!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

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 41 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 41 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@LibbyBot
Copy link

@LibbyBot
Copy link

@LibbyBot
Copy link

@LibbyBot
Copy link

@jhanders34
Copy link
Member Author

!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

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

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.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 41 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 41 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@LibbyBot
Copy link

@LibbyBot
Copy link

@LibbyBot
Copy link

@LibbyBot
Copy link

https://libh-proxy1.fyre.ibm.com/cognitive/pipelineAnalysis.html?uuid=f81a7533-062d-4d22-93a9-8632ec7d5c3a - Open Liberty Personal Build RTC completed with errors/failures.

@wraschke
Copy link
Member

wraschke commented Dec 2, 2024

!libby@dev (view Open Liberty Delivery Requirements Verification - ❌ completed with errors/failures.)

@LibbyBot
Copy link

LibbyBot commented Dec 2, 2024

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 41 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@wraschke
Copy link
Member

wraschke commented Dec 2, 2024

@jhanders34 please bear with me while I run test pipelines to fix the bug we've been discussing in the past few weeks.

@wraschke
Copy link
Member

wraschke commented Dec 2, 2024

Please ignore! !libby (view Open Liberty Delivery Requirements Verification - ✅ completed successfully!)

@LibbyBot
Copy link

LibbyBot commented Dec 2, 2024

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 1 infrastructure code files were changed.

  • 41 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

- 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
@jhanders34
Copy link
Member Author

jhanders34 commented Dec 18, 2024

!build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 24 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 1 infrastructure code files were changed.

  • 6 test infrastructure code files were changed.

  • Test failures/errors in the build could be due to these changes.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@wraschke
Copy link
Member

wraschke commented Dec 19, 2024

Sorry, Jared, please ignore any resulting pipelines. This is for testing bug we discussed.
(view Open Liberty Delivery Requirements Verification - ✅ completed successfully!)

!libby@dev

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 24 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 1 infrastructure code files were changed.

  • 6 test infrastructure code files were changed.

  • Test failures/errors in the build could be due to these changes.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@jhanders34
Copy link
Member Author

jhanders34 commented Dec 20, 2024

!build
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
fat.buckets.to.run=all

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 24 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 1 infrastructure code files were changed.

  • 6 test infrastructure code files were changed.

  • Test failures/errors in the build could be due to these changes.

  • 40 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants