exceptionfactory opened a new pull request, #10525:
URL: https://github.com/apache/nifi/pull/10525

   # Summary
   
   [NIFI-15211](https://issues.apache.org/jira/browse/NIFI-15211) Updates the 
NiFi Registry JSON Web Token Key Identifier resolution strategy, building on 
changes in #10521 and moving the `kid` claim location to work with the 
`KeyLocator` interface.
   
   The `SigningKeyResolverAdapter` is deprecated, and previously provided the 
ability to retrieve the Key Identifier from a claim in the JWT payload. The 
`KeyLocator` limits access to the parsed JWT header, which did not contain the 
`kid` in the initial implementation. Changes now include setting the Key 
Identifier in the JWT header for subsequent resolution using the `KeyLocator` 
implementation.
   
   Additional changes include adding `nifi-registry-web-api` to set the set 
paths changed that trigger execution of the `integration-tests` workflow. 
Integration tests for NiFi Registry exercise JWT generation and verification, 
validating these changes. The change also removes two unused methods from the 
`JwtService` class.
   
   # Tracking
   
   Please complete the following tracking steps prior to pull request creation.
   
   ### Issue Tracking
   
   - [X] [Apache NiFi Jira](https://issues.apache.org/jira/browse/NIFI) issue 
created
   
   ### Pull Request Tracking
   
   - [X] Pull Request title starts with Apache NiFi Jira issue number, such as 
`NIFI-00000`
   - [X] Pull Request commit message starts with Apache NiFi Jira issue number, 
as such `NIFI-00000`
   
   ### Pull Request Formatting
   
   - [X] Pull Request based on current revision of the `main` branch
   - [X] Pull Request refers to a feature branch with one commit containing 
changes
   
   # Verification
   
   Please indicate the verification steps performed prior to pull request 
creation.
   
   ### Build
   
   - [X] Build completed using `./mvnw clean install -P contrib-check`
     - [X] JDK 21
     - [ ] JDK 25
   
   ### Licensing
   
   - [ ] New dependencies are compatible with the [Apache License 
2.0](https://apache.org/licenses/LICENSE-2.0) according to the [License 
Policy](https://www.apache.org/legal/resolved.html)
   - [ ] New dependencies are documented in applicable `LICENSE` and `NOTICE` 
files
   
   ### Documentation
   
   - [ ] Documentation formatting appears as expected in rendered files
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to