Setting attachHeadersAsQueryString flag to False breaks long polling

[devslam]

Describe the bug
I'm creating a socket with long polling and it works when the attachHeadersAsQueryString flag is set True. However, it breaks when the attachHeadersAsQueryString flag is set True. It sends a request once and does not poll again.

I suspect it's because we're not sending in the ?X-Atmosphere-Transport=long-polling query parameter when the flag is set to False
Atmosphere Info

• version: 2.4.30
• atmosphere.js version: 2.3.8-javascript

Expected behavior
I'm expecting long polling to work when I set the attachHeadersAsQueryString flag to False. However it does not. I'm expecting it to poll consistently. It sends a request once and does not poll again.

It's my understanding that setting the flag to False would still send in query parameters that are required for long polling to work; it would not send in "debug/informational" headers as a query string

Note: Long polling works when the attachHeadersAsQueryString flag is set to True

Screenshots
If you look at the screenshots here. The X-Atmosphere-Transport=long-polling query parameter isn't being passed in when the attachHeadersAsQueryString flag is set to True

Flag set to True

Flag set to False

Systems (please complete the following information):

• OS: macOS
• Browser name and version: Chrome 88
• Java version and distribution: Java 8
• Server name and version: Tomcat 7.0.94

Additional context
The reason why I'm setting the attachHeadersAsQueryString flag to False is because I don't think it's necessary to send in the following headers as query strings
X-Atmosphere-tracking-id=*************
X-Atmosphere-Framework=2.3.8-javascript
X-atmo-protocol=true

[jfarcand]

@devslam Are you able to reproduce with the latest version of Atmosphere? 2.4 is old...

[devslam]

@devslam Are you able to reproduce with the latest version of Atmosphere? 2.4 is old...

I'll try it. Though shouldn't I be trying to reproduce it with the latest version of the JS library?

[devslam]

I'm able to reproduce this with Atmosphere 2.6.4 and Atmosphere.js 3.0.6. It seems like the issue would be with the JS library b/c it's not sending the X-Atmosphere-Transport=long-polling query parameter when attachHeadersAsQueryString is set to false