Skip to content

fix(synkronus): return JSON content type for user list - #1028

Open
Grace256c wants to merge 2 commits into
OpenDataEnsemble:devfrom
Grace256c:fix/synkronus-cli-user-list-response
Open

Grace256c wants to merge 2 commits into
OpenDataEnsemble:devfrom
Grace256c:fix/synkronus-cli-user-list-response

Conversation

@Grace256c

Copy link
Copy Markdown
Contributor

What & Why

Fixes synk user list reporting a successful response as an API error.

The Synkronus GET /api/users handler returned a valid JSON array but did not set Content-Type: application/json. The generated CLI client only parses the HTTP 200 response into JSON200 when the response content type is JSON, so the successful response was left unparsed and treated as an API error.

This PR updates the server handler to return the correct JSON content type and adds regression coverage for both the server response and the CLI client parsing path.

Closes #1021.

Type of Change

  • Bug fix
  • New feature / enhancement
  • Refactor / cleanup
  • Documentation
  • Chore / other

Component(s) Changed

  • formulus (React Native mobile app)
  • formulus-formplayer (form WebView UI)
  • synkronus (Go backend)
  • synkronus-cli (command-line client)
  • synkronus-portal (web admin UI)
  • desktop (Tauri app)
  • packages (tokens / components)
  • Documentation / DevOps / CI

Testing

  • Added/updated tests
  • Manually tested
  • Not applicable

Checklist

@najuna-brian
najuna-brian requested a review from r0ssing October 25, 2026 09:47

This branch has not been deployed

No deployments
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.

[synkronus-cli] synk user list reports a successful response as an error

1 participant