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

Update invokefunction to return headers #2

Open
alexellis opened this issue Jan 14, 2019 · 3 comments
Open

Update invokefunction to return headers #2

alexellis opened this issue Jan 14, 2019 · 3 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@alexellis
Copy link
Member

Task:

  • Update invokefunction to return the response headers
  • Update the caller function named Invoke to return the body, statusCode and errors
  • Update example to print statusCode and body

Suggested by Simon Pezcler and Rishabh Gupta so that connectors can do better debug and potentially use the result for something.

Right now the result is abandoned.

@alexellis alexellis added help wanted Extra attention is needed good first issue Good for newcomers labels Jan 14, 2019
@zeerorg
Copy link

zeerorg commented Jan 19, 2019

We can use headers for logging in response to function invocations, it can be especially useful when connecting to outside services since.

@zeerorg
Copy link

zeerorg commented Jan 31, 2019

This issue should be closed.

@Supesharisuto
Copy link

is this the extra header params in

InvokeWithContext(ctx context.Context, topic string, message *[]byte, headers http.Header)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants