Skip to content
This repository has been archived by the owner on Jul 24, 2018. It is now read-only.

Fix processing messages from anitya #81

Merged
merged 1 commit into from
Jul 15, 2016
Merged

Fix processing messages from anitya #81

merged 1 commit into from
Jul 15, 2016

Conversation

pypingou
Copy link
Member

Messages such as: https://apps.fedoraproject.org/datagrepper/id?is_raw=true&size=extra-large&id=2016-fc73c74c-fc5a-4f65-8ae9-87145bae82e4
Do contain a 'project' key but that key has a value None so instead
of having the expected {}, we end up with a None object that has no
.get() method of course.
With this change, we do things in two steps but at least it should work.

@pypingou
Copy link
Member Author

I've hotfix this in prod, let's see if that fixes the error I've been seeing

Messages such as: https://apps.fedoraproject.org/datagrepper/id?is_raw=true&size=extra-large&id=2016-fc73c74c-fc5a-4f65-8ae9-87145bae82e4
Do contain a 'project' key but that key has a value `None` so instead
of having the expected `{}`, we end up with a `None` object that has no
`.get()` method of course.
With this change, we do things in two steps but at least it should work.
@puiterwijk
Copy link
Contributor

🚢 Looks good to me.

@pypingou
Copy link
Member Author

Thanks :)

@pypingou pypingou merged commit 5401130 into develop Jul 15, 2016
@pypingou pypingou deleted the fix_anitya_rules branch July 15, 2016 14:41
@sayanchowdhury
Copy link
Contributor

👍 for me too

@pypingou
Copy link
Member Author

Thanks as well :)

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

Successfully merging this pull request may close these issues.

3 participants