Filter out Cognito Prefixes#37
Conversation
|
That's a bug. Let me take a look at it this weekend. Invalid characters should be URL escaped so it should look like I'll make sure that works instead of adding in a special case for this one circumstance. Thanks for letting me know. |
|
So a Query String quote would suffice, right? |
|
I think a simple url escaping would be inadequate for this. I'm quite new to Go, but I believe It should be escaped based on this: Based on the source of the error being either: Both of them use this: I think any character that isn't accepted in this array could be replaced with "_" or url escaped. |
First of all: YUGE THANKS for this plugin
AWS Cognito creates a weird 'Cognito:username' prefix, which makes my life upstream (proxy) a burden. e.g. after adding a fmt.Println for headername on jwt.go @ line 135:
Later, that happens with proxy:
This PR replaces 'Cognito:' with 'Cognito_', so we can still capture it.