Skip to content

JSON sent by OpenAPI functions is automatically converted to camel-case #192

Closed
#310
Closed
JSON sent by OpenAPI functions is automatically converted to camel-case#192
#310
Assignees
cdavernas
Labels
app: runnerConcerns the Runnerpriority: highIndicates a high priority issuetype: bugSomething isn't workingtype: regressionIssue due to a regressionweight: 1An issue that has a very low development impact
Milestone
v0.2.0

Description

@cdavernas
cdavernas
opened on Aug 22, 2022

What happened:

JSON sent by OpenAPI functions is automatically converted to camel-case

What you expected to happen:

Synapse to respect the explicitly defined casing of JSON sent by OpenAPI functions

How to reproduce it:

Define non-camel cased arguments to an OpenAPI function expecting JSON

Discussed in #175 (reply in thread)

Activity

  1. cdavernas
    added
    type: bugSomething isn't working
    priority: highIndicates a high priority issue
    weight: 1An issue that has a very low development impact
    app: runnerConcerns the Runner
    on Aug 22, 2022
  2. cdavernas
    added this to the 0.2.0 milestone on Aug 22, 2022
  3. cdavernas
    self-assigned this
    on Aug 22, 2022

  4. cdavernas commented on Aug 23, 2022

    @cdavernas
    cdavernas
    on Aug 23, 2022
    MemberAuthor

    Actually, all Dynamic have their properties converted into camel case during serialization. This bug is related to the camel case contract resolver configured for Newtonsoft JSON.

  5. cdavernas commented on Aug 23, 2022

    @cdavernas
    cdavernas
    on Aug 23, 2022
    MemberAuthor

    Fixed by neuroglia-io/framework@70e47ea

  6. cdavernas
    closed this as completedon Aug 23, 2022
  7. cdavernas
    reopened this on Aug 23, 2022
  8. cdavernas
    added a commit that references this issue on Aug 23, 2022

    fix(Solution): Updated Neuroglia Framework packages to latest version

    a106254
  9. cdavernas
    closed this as completedin a106254on Aug 23, 2022
  10. cdavernas
    reopened this on Aug 24, 2022
  11. cdavernas
    added a commit that references this issue on Aug 24, 2022

    fix(Worker): Replaced all serialization calls with JsonConvert calls …

    035ec2d
  12. cdavernas
    closed this as completedin 035ec2don Aug 24, 2022

  13. cdavernas commented on Nov 7, 2022

    @cdavernas
    cdavernas
    on Nov 7, 2022
    MemberAuthor

    Regressed due to commit b76b81dd

  14. cdavernas
    added
    type: regressionIssue due to a regression
    on Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

app: runnerConcerns the Runnerpriority: highIndicates a high priority issuetype: bugSomething isn't workingtype: regressionIssue due to a regressionweight: 1An issue that has a very low development impact

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions