Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Strange payloads in notifications #3258

Open
kzangeli opened this issue Aug 1, 2018 · 4 comments
Open

Strange payloads in notifications #3258

kzangeli opened this issue Aug 1, 2018 · 4 comments

Comments

@kzangeli
Copy link
Member

kzangeli commented Aug 1, 2018

We have detected a few pretty strange notification payloads, for example in:
test/functionalTest/cases/0876_attribute_dates/attrs_dates_overridden_by_user_subs.test
... where identical payloads in notifications have different Content-Length.
The entire difference is probably whitespace, but we need to look into this.

@fgalan
Copy link
Member

fgalan commented Aug 3, 2018

The change we need to do in that file was:

-Content-Length: 262
+Content-Length: REGEX((400|262))

@fgalan
Copy link
Member

fgalan commented Aug 3, 2018

Additionally in test/functionalTest/cases/1048_subscription_cache/three_subscriptions.test we need to change:

-Content-Length: 22REGEX(\d)
+Content-Length: REGEX(\d+)

although the one we intially used, 22REGEX(\d), was also pretty weird...

@fgalan
Copy link
Member

fgalan commented Aug 3, 2018

The PR from which the modifications come from, for reference: apinf#34

@kzangeli
Copy link
Member Author

kzangeli commented Aug 9, 2018

One more of these strange content-Lengths of payloads:
0876_entity_dates/entity_dates_overridden_by_user_subs.test, PR apinf#35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants