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

Improve fetching of pool offline metadata #1276

Merged
merged 5 commits into from
Oct 6, 2022
Merged

Conversation

erikd
Copy link
Contributor

@erikd erikd commented Oct 3, 2022

No description provided.

Since we now check that the content-type is some form of JSON and we
return a `PoolOfflineData` struct, we also rename the function to
`httpGetPoolOfflineData`.

Closes: #1270
@erikd erikd requested review from a team and kderme as code owners October 3, 2022 23:54
erikd added 3 commits October 4, 2022 11:29
Can be run in two ways:
```
cabal run -- http-get-json-metadata <url>
cabal run -- http-get-json-metadata <url> <hash>
```
The first of the above two commands does not check the correctness of
the metadata hash (eg as specified in the `PoolMetadataRef` table).
The second of the above two commands does check the hash. If the user
does not know the hash, they can provide a dummy hash like "00". The
hash check will then fail, providing the actual hash. If this actual
hash is then supplied again it should pass.

On success the program will print "Success" and the JSON. On failure it
prints the error.
Just a debug/test program that pulls PoolMetadataRef info from the
database and retries them.
@erikd erikd force-pushed the erikd/http-offline-data branch from b667bf8 to 38a8aaa Compare October 4, 2022 00:31
@erikd erikd merged commit 339a349 into master Oct 6, 2022
@iohk-bors iohk-bors bot deleted the erikd/http-offline-data branch October 6, 2022 02:02
reqlez pushed a commit to cardano-bsd-alliance/cardano-db-sync that referenced this pull request Oct 7, 2022
…offline-data

Improve fetching of pool offline metadata
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.

1 participant