[PR] Reapply "KNOX-3279 - REST Catalog dispatch implementation for including configurable metadata as outbound request headers (#1182)" (#1188) [knox]

["pzampino (via GitHub)" <gi...@apache.org>]

pzampino opened a new pull request, #1191:
URL: https://github.com/apache/knox/pull/1191

   
   This reverts commit a695134ec8e6fdfd46bcade58e25235f538280b5, restoring the implementation of KNOX-3279 plus fixes for the issues which caused it to be reverted.
   
   ## What changes were proposed in this pull request?
   
   Implemented ICEBERG-REST custom dispatches, adding the ability to configure token metadata which ought to be manifest as headers in requests dispatched to the service. Also made the prefix for these header names configurable.
   
   ## How was this patch tested?
   
   mvn -Ppackage,release clean verify (includes new unit tests for the dispatches)
   
   ant install-test-home
   Manual testing using two topologies and a series of curl commands:
   cc-test: LDAP authenticated CLIENTID service topology
   rc-test: JWTProvider authenticated ICEBERG-REST service topology
   
   Acquire the client_id and client_secret:
   curl -ivku admin 'https://localhost:8443/gateway/cc-test/clientid/api/v1/oauth/credentials'
   
   Use the client_id/client_secret to exercise the custom dispatch
   curl -ivk -H 'Content-Type: application/x-www-form-urlencoded' -d 'client_secret=**<CLIENT_SECRET>**=&grant_type=client_credentials&client_id=**<CLIENT_ID>**' 'https://localhost:8443/gateway/rc-test/iceberg-rest/api/v1/'


-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Reapply "KNOX-3279 - REST Catalog dispatch implementation for including configurable metadata as outbound request headers (#1182)" (#1188) [knox]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #1191:
URL: https://github.com/apache/knox/pull/1191#issuecomment-4145831039

   ## Test Results
   7 tests   7 ✅  1s ⏱️
   1 suites  0 💤
   1 files    0 ❌
   
   Results for commit cd08369a.
   
   [test-results]:data:application/gzip;base64,H4sIAHINx2kC/1WMSQ6DMBAEv4J8zsETZBjymciMjTQK4MjLKcrfYwiLuXVVS/URA482iEcFt0qExPEAk7yO7OYN8xGXq933MySiq3jxOwt5iEHzeBHWe+c349O895ZZ5v581lYuYiuXLXLTxDGDICOxbjoNrSYA7O61BdX30khjlUKwSEgKSXx/iDfNWfsAAAA=
   


-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Reapply "KNOX-3279 - REST Catalog dispatch implementation for including configurable metadata as outbound request headers (#1182)" (#1188) [knox]

["pzampino (via GitHub)" <gi...@apache.org>]

pzampino merged PR #1191:
URL: https://github.com/apache/knox/pull/1191


-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org