Conversation
|
I agree to this solution.
The approach from @willoucom made that work! |
|
AFAIK header which looks like below is invalid syntax. |
|
We occured the same issue in another scenario. I agree that this is basically not the problem of the library, but it would add some resilience if the lib would just take the first header starting with "Basic". |
|
I agree with @tuupola , having 2 different authorization is invalid. However, I think the problem with some loadbalancers (e.g. amazon/scaleway/azure) duplicating the header can lead to a lot of problems for developers wanting to use this library (which I consider great). What do you think about this alternative? |
Fixes #89
Since getHeaderLine() can return a coma separated string, i add an explode and a foreach.
Updated tests to reflect the change