Skip to content

Bump aeson to version 2.0 and don't use HashMap #184

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

Closed
wants to merge 2 commits into from

Conversation

georgefst
Copy link
Contributor

Closes #51.

I had a brief look at this after it came up this afternoon, and it turned out there isn't much that needs doing.

Marked as a draft since --allow-newer isn't ideal.

type: git
location: https://github.com/hasufell/aeson-pretty
tag: e902ab866bb41d990b66af3644aeb352ff7aaf6f
--sha256: 08pc8043ps0zn0nkxy9mabdzglcn7ywgybi3x8az399l0qgjwzv8
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brprice
Copy link
Contributor

brprice commented Oct 21, 2021

Looks like hydra is failing to build openapi3 because of the aeson bump. Does not look like anyone has fixed it upstream yet.

@georgefst
Copy link
Contributor Author

Looks like hydra is failing to build openapi3 because of the aeson bump. Does not look like anyone has fixed it upstream yet.

Ah, not surprised there's something. I only got as far as checking the core primer library built.

@@ -12,3 +12,21 @@ package *

package primer
test-options: "--size-cutoff=32768"

-- https://github.com/informatikr/aeson-pretty/pull/37
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR has been merged and released as aeson-pretty-0.8.9, so the override is no longer necessary.

@georgefst
Copy link
Contributor Author

Closing in favour of #244, which will add the correct lower bound on Aeson. The overrides here should no longer be necessary.

@georgefst georgefst closed this Feb 7, 2022
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

Successfully merging this pull request may close these issues.

Remote exploit in aeson
2 participants