SG as library: Vars config, filter panic: interface conversion: interface {} is int, not string about graphjin

[dosco]

Fixed in 9c19db2

[frederikhors]

I found the issue, reading your code here: https://github.com/dosco/super-graph/blob/master/core/args.go#L33.

I was passing an int and not a string for UserIDKey value in context, with this code:

ctx := context.WithValue(r.Context(), core.UserIDKey, authentication.MyID()) // int

Now I'm using this instead:

ctx := context.WithValue(r.Context(), core.UserIDKey, strconv.Itoa(authentication.MyID()))

At first I didn't notice the error because nowhere is it written that it must be a string.

Shall we create a page for config and "using SuperGraph as a library" related?

Can we specify it in code docs? Do you agree, @dosco?

[dosco]

Thanks that would be very valuable if you could document your experience with using Super Graph as a library. You can add a page to this folder and i'll include it on the website. https://github.com/dosco/super-graph/tree/master/docs/website/docs

[frederikhors]

Yes I'll create a page in a few hours.

For this problem, can we use both string and int for user_id?

Here the code (https://github.com/dosco/super-graph/blob/master/core/args.go#L29-L36):

for i, p := range params {
  switch p.Name {
  case "user_id":
    if v := c.Value(UserIDKey); v != nil {
      vars[i] = v.(string) // maybe a case for int too?
    } else {
      return nil, argErr(p)
    }
...

[dosco]

That should work.