Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

feat: add initial tile proxying #9

Merged
merged 4 commits into from
Feb 9, 2021
Merged

feat: add initial tile proxying #9

merged 4 commits into from
Feb 9, 2021

Conversation

pjenvey
Copy link
Member

@pjenvey pjenvey commented Feb 6, 2021

Description

  • proxy to ADM_ENDPOINT_URL
  • add basic validation and mapping to AdM's region ip from the country
    param
  • add an inital integration test w/ an AdM Tile API mock
  • fix circleci/Dockerfile configs

Note there's a few TODOS/XXX's here, I'll log separate issues for each of them shortly

Issue(s)

Closes #2, #3, #5

@pjenvey
feat: add initial tile proxying
2cc96d0 
- proxy to ADM_ENDPOINT_URL
- add basic validation and mapping to AdM's region ip from the country
param
- add an inital integration test w/ an AdM Tile API mock
- fix circleci/Dockerfile configs

Closes #2, #3, #5
@pjenvey pjenvey requested review from ncloudioj and a team February 6, 2021 01:40
@pjenvey pjenvey mentioned this pull request Feb 9, 2021
Merged
jrconlin
jrconlin approved these changes Feb 9, 2021
View reviewed changes
src/error.rs Outdated Show resolved Hide resolved
src/web/mod.rs Outdated Show resolved Hide resolved
@jrconlin jrconlin merged commit 71b9df3 into main Feb 9, 2021
This was referenced Feb 9, 2021
Closed
Closed
@jrconlin jrconlin deleted the feat/3 branch February 23, 2021 17:38
@pjenvey pjenvey mentioned this pull request Jun 2, 2021
Closed
Trinaa pushed a commit that referenced this pull request Jun 13, 2022
@hackebrot
Merge pull request #9 from mozilla-services/add-project-documentation
e675572 
Add high-level documentation for the integration tests 📝
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jrconlin jrconlin jrconlin approved these changes

@ncloudioj ncloudioj Awaiting requested review from ncloudioj

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Setup Circle CI integration
2 participants
@pjenvey @jrconlin