search

epacke / BigIPReport

Overview of your loadbalancer configuration
https://loadbalancing.se
8 stars 2 forks source link

getMonitorRequestParameters does not parse headers for curl #154

Closed timriker closed 4 years ago

timriker commented 5 years ago

We're looking for '\r\n' instead of '\\r\\n'.

timriker commented 5 years ago

Hmm. It looks like either check might work. Perhap we should handle both? Testing.

timriker commented 5 years ago

Looks like there was a hot fix that caused the slashes to get duplicated. We'll fix them on our end. Reverting this part of the change.

timriker commented 5 years ago

Well, isn't this ugly. BIG-IP accepts either \r\n (correct single backslash) or \\r\\n (incorrect double backslash) and in the second case it looks like it adds another two \r\n pairs in case the user forgot them. Let's parse on either the Good \r\n or the Bad \\r\\n formats.

epacke commented 4 years ago

Ewwww, ugly indeed!