Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ALPS profile does not reflect jackson customization (e.g. enableEnumTranslation=true) [DATAREST-683] #1056

Closed
spring-projects-issues opened this issue Sep 23, 2015 · 2 comments
Assignees
Labels
type: bug

Comments

@spring-projects-issues
Copy link

@spring-projects-issues spring-projects-issues commented Sep 23, 2015

Oliver Trosien opened DATAREST-683 and commented

Found while trying to document the enums of my REST API with ALPS.
See example including failing test here: https://github.com/otrosien/spring-alps-bug

The alps profile starts like below, but seems like it uses a differently configured jackson mapper, as the main one is configured to "enableEnumTranslation" ("Payment expected" instead of "PAYMENT_EXPECTED").

{
  "version" : "1.0",
  "descriptors" : [ {
    "id" : "order-representation",
    "href" : "http://localhost/profile/orders",
    "descriptors" : [ {
      "name" : "status",
      "doc" : {
        "value" : "PAYMENT_EXPECTED, PAID",
        "format" : "TEXT"
      },
      "type" : "SEMANTIC"
    } ]

Affects: 2.4 GA (Gosling)

Referenced from: commits 35a86f7, cc7bc7a, 1f133fb, 75f1143

Backported to: 2.4.1 (Gosling SR1)

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Sep 23, 2015

Oliver Trosien commented

maybe related to DATAREST-676.

Another question: Is the ALPS JSON response actually valid ALPS? For example the alps-console project cannot read this file ( https://github.com/tkawa/alps-console ), and I did notice some differences, like missing "alps" root element, or "descriptors" instead of "descriptor"

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Nov 6, 2015

Oliver Drotbohm commented

That's both fixed for Hopper and Gosling

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug
Projects
None yet
Development

No branches or pull requests

2 participants