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

Fix handling of the consolidate-redundant-resources modification that goes live Dec 6 #171

Closed
justincy opened this issue Nov 10, 2016 · 1 comment

Comments

@justincy
Copy link
Contributor

justincy commented Nov 10, 2016

When the consolidate-redundant-resources is live it returns a 303 redirect from Person With Relationships, Spouse Relationships, Child Relationships, Parent Relationships, Source References, Discussion References, and Memory References. 3xx redirects have always been problematic for the SDK because Chrome doesn't copy all portions of the original request include the Accept header so on the subsequent request the API defaults to an XML response.

The good news is we can add the X-Expect-Override header on requests to those resources and manually handle the redirect. We will publish a new version of the SDK ASAP that does this.

The bad news is when the pending modification is permanently activated on Dec 6, all old versions of the SDK will not be able to handle requests to any of the deprecated resources. I think.

@justincy
Copy link
Contributor Author

List of deprecated SDK methods:

  • getPersonWithRelationships()
  • Person.getSpouseRelationships()
  • Person.getParentRelationships()
  • Person.getChildRelationships()
  • getSourceRefs()
  • Person.getSourceRefs()
  • ChildAndParents.getSourceRefs()
  • Couple.getSourceRefs()
  • getPersonDiscussionRefs()
  • Person.getDiscussionRefs()
  • getMemoryPersonaRefs()

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

No branches or pull requests

1 participant