You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "zgzyq (via GitHub)" <gi...@apache.org> on 2026/09/01 07:07:44 UTC

[PR] [fix][fn] Return descriptive errors for unknown JSON properties [pulsar]

zgzyq opened a new pull request, #26441:
URL: https://github.com/apache/pulsar/pull/26441

   Fixes #18253
   
   ### Motivation
   
   The Functions Worker APIs return a generic HTTP 400 response when a multipart JSON configuration contains an unknown property. Pulsar already has an `UnrecognizedPropertyExceptionMapper` that produces a descriptive error, but the Functions Worker resource configurations do not register it.
   
   ### Modifications
   
   - Register `UnrecognizedPropertyExceptionMapper` for the Functions Worker V2 APIs.
   - Register `UnrecognizedPropertyExceptionMapper` for the Functions Worker V3 APIs.
   - Add a regression test covering both resource sets.
   
   ### Verifying this change
   
   This change added tests and can be verified as follows:
   
   - `./gradlew :pulsar-functions:pulsar-functions-worker:test --tests "org.apache.pulsar.functions.worker.rest.ResourcesTest" -PtestRetryCount=0`
   - `./gradlew :pulsar-functions:pulsar-functions-worker:checkstyleMain :pulsar-functions:pulsar-functions-worker:checkstyleTest`
   
   ### Does this pull request potentially affect one of the following parts:
   
   - [ ] Dependencies (add or upgrade a dependency)
   - [ ] The public API
   - [ ] The schema
   - [ ] The default values of configurations
   - [ ] The threading model
   - [ ] The binary protocol
   - [x] The REST endpoints
   - [ ] The admin CLI options
   - [ ] The metrics
   - [ ] Anything that affects deployment


-- 
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: commits-unsubscribe@pulsar.apache.org

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

Re: [PR] [fix][fn] Return descriptive errors for unknown JSON properties [pulsar]

Posted by "merlimat (via GitHub)" <gi...@apache.org>.
merlimat merged PR #26441:
URL: https://github.com/apache/pulsar/pull/26441


-- 
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: commits-unsubscribe@pulsar.apache.org

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