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

AppSync in the response resolver to support "Set-cookie", and supporing httponly, maxAge, Secure, SameSite parameters #461

Open
2 tasks
yanlin96 opened this issue May 5, 2022 · 1 comment
Labels
feature-request New feature or request GraphQL

Comments

@yanlin96
Copy link

yanlin96 commented May 5, 2022

Is this related to a new or existing framework?

No, should be a AppSync only

Is this related to a new or existing API?

GraphQL API

Is this related to another service?

Cloudfront maybe

Describe the feature you'd like to request

Firstly, AppSync could have the ability to read the httponly cookie through the request headers;

Secondly, there is a use case that we want to set a httponly cookie in AppSync response resolver and send it back to the front end and reflect it in the browser. Currently, we have tried to use "$util.http.addResponseHeader("set-cookie":xxxxxxxxx)", however, after I set up a Nginx server with the same domain as AppSync locally, I could not see the cookie there in the response header.

Describe the solution you'd like

something similar like "$util.http.addResponseHeader("set-cookie":xxxxxxxxx)" with more parameters supported like httponly, secure, maxAge to put the set-cookie in the response header. And all resolvers should support that. Also need to considering integration with cloudfront and route53 maybe.

Describe alternatives you've considered

None

Additional context

No response

Is this something that you'd be interested in working on?

  • 👋 I may be able to implement this feature request
  • ⚠️ This feature might incur a breaking change
@yanlin96 yanlin96 added the feature-request New feature or request label May 5, 2022
@benm5678
Copy link

benm5678 commented May 5, 2023

Yes, completely agree...if it can already set other headers, is there some technical challenge to allow us to also return set-cookie?

@stocaaro stocaaro transferred this issue from aws-amplify/amplify-js Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or request GraphQL
Projects
None yet
Development

No branches or pull requests

3 participants