Y24-191: Add API keys to SS requests

sanger / lighthouse

API to search through data provided by Lighthouse Labs

MIT License

2 stars 1 forks source link

Y24-191: Add API keys to SS requests #958

Closed sdjmchattie closed 4 months ago

sdjmchattie commented 4 months ago

Closes #950

Changes proposed in this pull request

Add the SS authentication headers to all requests for SS.
Add tests to ensure the headers are always present.

Instructions for Reviewers

[All PRs] - Confirm PR template filled
[Feature Branches] - Review code

sdjmchattie commented 4 months ago

So looks like this was mostly implemented already (adding API key to requests), but just a couple were missed?

Yep that's right. The request header addition should probably have been somewhere more central, but this is a legacy product as far as Tom keeps telling us, so I went for the minimal viable solution.