Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

Web.pm: Switch fallback server to ddh1.duckduckgo.com #100

Merged
merged 1 commit into from Jul 29, 2014
Merged

Conversation

jagtalon
Copy link
Member

I would like to change the default fallback server to DDH1 so that people can have access to private APIs.

We sometimes get pull requests from partners that don't have signup pages for APIs. When this happens, only people in DDG have the capability to actually check the API. But when we point it to DDH1, I can deploy the endpoint there and people will be able to play with it on their own DuckPAN installations.

Flow:

  1. DuckPAN checks if the env variable is available.
  2. If it is, it makes a request to the API endpoint.
  3. If it isn't, it makes a request to ddh1.

Here's a PR where this can be applied immediately: duckduckgo/zeroclickinfo-spice#983

Here it is in action:
adsfasdfdsaf

CC @moollaza @nilnilnil @malbin @russellholt

I would like to change the default fallback server to DDH1 so that people can have access to private APIs.

We sometimes get pull requests from partners that don't have signup pages for APIs. When this happens, only people in DDG have the capability to actually check the API. But when we point it to DDH1, I can deploy the endpoint there and people will be able to play with it on their own DuckPAN installations.

Flow:

1. DuckPAN checks if the env variable is available.
2. If it is, it makes a request to the API endpoint.
3. If it isn't, it makes a request to ddh1.
@moollaza
Copy link
Member

I'm fine with this, LGTM 👍

moollaza added a commit that referenced this pull request Jul 29, 2014
Web.pm: Switch fallback server to ddh1.duckduckgo.com
@moollaza moollaza merged commit 9a1991e into master Jul 29, 2014
@jagtalon jagtalon deleted the jag/api-keys branch August 1, 2014 21:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants