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

feat: Added support for easy connection to an AtSecondaryReverseProxy #194

Merged
merged 2 commits into from
Jul 11, 2022

Conversation

gkc
Copy link
Contributor

@gkc gkc commented Jul 7, 2022

- What I did
Feat: Enhanced SecondaryUrlFinder so by convention it will return a secondary address of <proxy domain>:<'root' port> when rootDomain is supplied as 'proxy:<proxy domain>' and 'rootPort' is supplied as the port on which the reverse proxy is listening

e.g. rootDomain of 'proxy:vip.ve.atsign.zone' and rootPort of '8443' will result in all secondary lookups returning the same address, being the address of the proxy : vip.ve.atsign.zone:8443 in this example

Test: Added tests to verify behaviour of CacheableSecondaryAddressFinder with rootDomain set to 'proxy:'

- How to verify it
Tests should pass. For end-to-end full verification, see instructions here for setting up an AtSecondaryServer reverse proxy

- Description for the changelog
Feat: Added support for easy connection to an AtSecondaryReverseProxy

Feat: Enhanced SecondaryUrlFinder so by convention it will return a secondary address of <proxy domain>:<'root' port> when rootDomain is supplied as 'proxy:<proxy domain>' and 'rootPort' is supplied as the port on which the reverse proxy is listening

Test: Added tests to verify behaviour of CacheableSecondaryAddressFinder with rootDomain set to 'proxy:<something>'
@gkc gkc requested a review from VJag July 7, 2022 14:31
@gkc gkc changed the title Feat: Added support for easy connection to an AtSecondaryReverseProxy feat: Added support for easy connection to an AtSecondaryReverseProxy Jul 7, 2022
Feat: Enhanced SecondaryUrlFinder so by convention it will return a secondary address of <proxy domain>:<'root' port> when rootDomain is supplied as 'proxy:<proxy domain>' and 'rootPort' is supplied as the port on which the reverse proxy is listening

Test: Added tests to verify behaviour of CacheableSecondaryAddressFinder with rootDomain set to 'proxy:<something>'
@gkc gkc merged commit c508055 into trunk Jul 11, 2022
@gkc gkc deleted the gkc_add_reverse_proxy_support_to_find_secondary branch July 11, 2022 13:38
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.

2 participants