You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The current proxycontext is limited(i.e. UserData is in use and we can't use it to attach extra data).
Refactoring the context to use the golang's context should solve a lot of issues(or potential issues) and to bring consistency with the community's context.
The text was updated successfully, but these errors were encountered:
I'm not really sure why UserData isn't good enough, you can attach whatever
you want to it. If you want to attach something to it, you can simply attach
struct {
somethingA A
somethingB B
}
Or am I missing something?
What exactly do you purpose?
On Tue, Jun 27, 2017 at 12:58 PM, Almog Baku ***@***.***> wrote:
The current proxycontext is limited(i.e. UserData is in use and we can't
use it to attach extra data).
Refactoring the context to use the golang's context should solve a lot of
issues(or potential issues) and to bring consistency with the community's
context.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#227>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAP4ovLVjFCmrKxpLmHH8mYMwBr3i_CXks5sINIqgaJpZM4OGZx_>
.
My objective is to put on the context the information about the user.
Also, I suggest we'll move to the generic context package in order to allow compatibility with different packages.
The current implementation of UserData does not allow different packages to use this field because you need to put there a common struct that every package need to know how to put the information in.
Usage of context package solves that by having a common interface for that.
The current
proxycontext
is limited(i.e.UserData
is in use and we can't use it to attach extra data).Refactoring the context to use the golang's context should solve a lot of issues(or potential issues) and to bring consistency with the community's context.
The text was updated successfully, but these errors were encountered: