feat: GA4GH passport demonstration #240
Conversation
| let brokers = HashMap::from([ | ||
| ("https://broker-europe.aai.dev.umccr.org", DecodingKey::from_rsa_components("n5POaIPBp04XYkujy7ILkeYpuqPtzRz6fWNFZy7fR6qPLycP4aANFA2xRjr5YP1XXRnm7Jg23gmSbGYFNlnKDRNf67PM53L9Afx56DAUufH0vAISOq2e-i2P4aWZCGcc-d-8tmNTQ3FFcS2wD3bwUsVG2uLXVcdHvmvbTVVIXYxNiznXLk3sNBjuL40VIKEK_x8KSX04_0_x07KKFW1rqj1sguzBeF-NJRTGKuplFEwVM5TxAXRNQe1VeC3_TAEK4PRD8bzzFBz3y-fyovlppfjeOEbIlLT4mafzD130dlINw4xdaLQIPkQb8UE8O-XNKUzguSdUOw0TYB49mFIm5w", e).unwrap()), | ||
| ("https://broker-australia.aai.dev.umccr.org", DecodingKey::from_rsa_components("q-mBB5jBJCCc68ALWF_A-zOM4S5gsKPB6qeFYWe_uzkfWf-jcSCHRrRCsMRzflVvZbz3mbBmqp8FVOnEWQe62x0qdNSZRUmRWkeBKhi0yxjXbKV7e11Sv5XWxxGhYL-gYzJXqQLR9T8ZfcDeQvEtobznm51VkZ1UgD6QogjtpCK-LL3t5NK2wS5lZO3K5GM4spbnXOLbbUU0sRHKujkYa6frY71i3EAs_nrzkTRmT0I_QkE24XlGRh0zbM67pW1n9SKHsEpFIEzXy3ebBfHhaWKdxY4qhGqbOvft-rgNGGAXPEkKbygIcE0Uif1DvaNHD6KDAeP0DGJVr3qCKQ4Naw", e).unwrap()), | ||
| ]); |
There was a problem hiding this comment.
yes those are keys! they are public keys for known RSA keys though..
|
asking for review not for merging purposes.. just let me know anything I've done wildly wrong. If ok - can we have this branch deployed to UoM demo? |
|
Quite hardcore/hardcoded changes not to be merged to main, but I see your intent. Changes look fine within those premises, so I'd say go ahead and deploy to UoM demo 🚀 |
|
Yes to get rid of the embedded keys we'd need to instigate a new config mechanism - and probably move the either auth into some sort of plugin architecture. Which is a bit more than I want for this demo. I don't know how to deploy to demo btw. (I mean, is it literally the CDK in deploy?) |
files for ga4gh demo (just for local test)
|
Thanks for this Andrew! Closing this for now as we'll not be merging and keeping the branch around so that we can revisit this and implement/merge once GA4GH impls (and elsewhere upstream) are a bit more stable? Probably worth looking at it again once you're back from Sweden? |
|
Yes no problems - I am going to grab the code and possibly put it in a standalone GA4GH repo of "passport decoders" (in different languages). So it won't need to have any htsget tie in.. |
Not for merging.
Just a quick demonstration that we can deploy of passports as Auth headers.