Skip to content

feat: add message param and error responses to createTicket#632

Merged
biilmann merged 6 commits intomasterfrom
feat/oauth-tickets-message-and-errors
Mar 11, 2026
Merged

feat: add message param and error responses to createTicket#632
biilmann merged 6 commits intomasterfrom
feat/oauth-tickets-message-and-errors

Conversation

@biilmann
Copy link
Copy Markdown
Member

Summary

  • Add message parameter to the createTicket endpoint request body for OAuth ticket creation
  • Add 401 (Unauthorized) and 422 (Unprocessable Entity) error response schemas to createTicket
  • Bump version to 2.51.0 to match release
  • Rebuild Go client with updated models

Test plan

  • CI passes (npm test — lint, build, unit tests)
  • Go client compiles with new models
  • Verify createTicket endpoint accepts message in request body

🤖 Generated with Claude Code

biilmann and others added 5 commits March 10, 2026 17:51
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The message parameter was being sent as a query parameter, which leaks
potentially sensitive content in URLs and server logs. Move it to the
POST request body by updating the OpenAPI spec and regenerating the
Go client.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The 401 and 422 response types were not deserializing the error body,
dropping the structured error payload. Add error schema references in
the swagger spec and regenerate so the response types expose
Payload *models.Error and consume the response body.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@biilmann biilmann requested review from a team as code owners March 11, 2026 21:04
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 11, 2026

Deploy Preview for open-api ready!

Name Link
🔨 Latest commit 2ea690d
🔍 Latest deploy log https://app.netlify.com/projects/open-api/deploys/69b1d9c20fe0470008e465fe
😎 Deploy Preview https://deploy-preview-632--open-api.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 74d245da-fcca-4ce2-b261-ba28af03cbb0

📥 Commits

Reviewing files that changed from the base of the PR and between 6c485a0 and 6ded4ef.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (5)
  • go/models/create_ticket_params_body.go
  • go/plumbing/operations/create_ticket_parameters.go
  • go/plumbing/operations/create_ticket_responses.go
  • package.json
  • swagger.yml

📝 Walkthrough

Summary by CodeRabbit

Release Notes

  • New Features

    • The POST /oauth/tickets endpoint now accepts a message field in the request body, allowing clients to create tickets with enhanced messaging capabilities.
  • Bug Fixes

    • Improved error handling with proper HTTP response codes: 401 Unauthorized when OAuth application is not found, and 422 Unprocessable Entity for validation errors.

Walkthrough

This pull request adds request body support and error response handling to the POST /oauth/tickets endpoint. A new Go model CreateTicketParamsBody is introduced containing a Message field. The CreateTicketParams operation is updated with a Body field and supporting methods (WithBody, SetBody) to handle request bodies. Response processing is extended to handle 401 (Unauthorized) and 422 (Unprocessable Entity) error cases with dedicated response types. The API version is incremented from 2.50.0 to 2.51.0 in package manifests and the OpenAPI specification is updated to document the request body schema and new error responses.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • sean-roberts
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main changes: adding a message parameter and error responses to the createTicket endpoint.
Description check ✅ Passed The description clearly outlines the key objectives—adding message parameter, error responses, version bump, and Go client rebuild—and is directly related to the changeset.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/oauth-tickets-message-and-errors

Comment @coderabbitai help to get the list of available commands and usage tips.

@biilmann biilmann merged commit 4b3a79a into master Mar 11, 2026
15 checks passed
@biilmann biilmann deleted the feat/oauth-tickets-message-and-errors branch March 11, 2026 21:35
biilmann pushed a commit that referenced this pull request Mar 11, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.51.0](v2.50.0...v2.51.0)
(2026-03-11)


### Features

* add message param and error responses to createTicket
([#630](#630))
([6c485a0](6c485a0))
* add message param and error responses to createTicket
([#632](#632))
([4b3a79a](4b3a79a))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: token-generator-app[bot] <82042599+token-generator-app[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants