You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "PG1204 (via GitHub)" <gi...@apache.org> on 2026/07/16 20:06:19 UTC

[I] Computing-unit getOwner returns HTTP 500 for a nonexistent unit instead of 404 [texera]

PG1204 opened a new issue, #6475:
URL: https://github.com/apache/texera/issues/6475

   ### What happened?
   
   ### What happened?
   ComputingUnitAccessResource.getOwner (GET /access/computing-unit/owner/{cuid}) throws a plain IllegalArgumentException when the unit doesn't exist:
   val unit = workflowComputingUnitDao.fetchOneByCuid(cuid)
   if (unit == null) {
     throw new IllegalArgumentException("Computing unit does not exist")
   }
   
   The service registers no ExceptionMapper for IllegalArgumentException (only UnauthorizedExceptionMapper), so it falls through to Dropwizard's default LoggingExceptionMapper and surfaces as an opaque HTTP 500 ("There was an error processing your request…"), rather than a proper client error.
   
   This is the same unmapped-exception pattern fixed for grantAccess/revokeAccess in #6446; getOwner was left out of that PR's scope.
   
   Expected: requesting the owner of a nonexistent cuid returns a 404 (NotFoundException) with a clear message. Matches the JAX-RS convention now used by the other methods in this resource and by DatasetAccessResource/WorkflowAccessResource.
   
   
   ### How to reproduce?
   
   With computing-unit sharing enabled, call GET /access/computing-unit/owner/{cuid} with a cuid that doesn't exist → HTTP 500 instead of 404.
   
   ### Version/Branch
   
   1.3.0-incubating-SNAPSHOT (main)
   
   ### Commit Hash (Optional)
   
   _No response_
   
   ### What browsers are you seeing the problem on?
   
   _No response_
   
   ### Relevant log output
   
   ```shell
   java.lang.IllegalArgumentException: Computing unit does not exist
   => HTTP 500 (default LoggingExceptionMapper)
   ```


-- 
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: notifications-unsubscribe@texera.apache.org.apache.org

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

Re: [I] Computing-unit getOwner returns HTTP 500 for a nonexistent unit instead of 404 [texera]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #6475:
URL: https://github.com/apache/texera/issues/6475#issuecomment-4996114315

   <!-- texera:template-compliance -->
   👋 Thanks for opening this issue, @PG1204!
   
   It looks like the issue description doesn't quite follow our template yet:
   
   - The **What happened?** section is empty; please fill it in.
   
   Filling out the template helps reviewers understand and triage your contribution faster. Please edit the description to complete it. This message will disappear automatically once the template is followed.
   
   You can find the template prompts by editing the description, or see [CONTRIBUTING.md](https://github.com/apache/texera/blob/main/CONTRIBUTING.md) for the full contribution flow.
   


-- 
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: notifications-unsubscribe@texera.apache.org

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

Re: [I] Computing-unit getOwner returns HTTP 500 for a nonexistent unit instead of 404 [texera]

Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on issue #6475:
URL: https://github.com/apache/texera/issues/6475#issuecomment-4996117718

   /take


-- 
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: notifications-unsubscribe@texera.apache.org

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

Re: [I] Computing-unit getOwner returns HTTP 500 for a nonexistent unit instead of 404 [texera]

Posted by "aglinxinyuan (via GitHub)" <gi...@apache.org>.
aglinxinyuan closed issue #6475: Computing-unit getOwner returns HTTP 500 for a nonexistent unit instead of 404
URL: https://github.com/apache/texera/issues/6475


-- 
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: notifications-unsubscribe@texera.apache.org

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

Re: [I] Computing-unit getOwner returns HTTP 500 for a nonexistent unit instead of 404 [texera]

Posted by "PG1204 (via GitHub)" <gi...@apache.org>.
PG1204 commented on issue #6475:
URL: https://github.com/apache/texera/issues/6475#issuecomment-4996103049

   /take


-- 
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: notifications-unsubscribe@texera.apache.org

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