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

QueryString formatting #57

Closed
ksonda opened this issue Jun 17, 2020 · 2 comments
Closed

QueryString formatting #57

ksonda opened this issue Jun 17, 2020 · 2 comments

Comments

@ksonda
Copy link
Member

ksonda commented Jun 17, 2020

We've been asking for QueryStrings for content types that look like f?=.*

For some reason, this means that GET requests without ?f=anything are redirected to ones with ?f= appended. I don't know if this is bad, but it seems unclean.

If you change the QueryString to f=.* you get normal behavior

For example: https://geoconnex.us/ref/hu02/02 -> 303https://info.geoconnex.us/collections/hu02/items/02?f=

changing f?=.* to f=.* : https://geoconnex.us/ref/hu02/01 -> 303https://info.geoconnex.us/collections/hu02/items/01

I don't know if this breaks anything. I wonder if it has to do with the pygeoapi index issues we have been having (because what google infers the canonical URL to be based on all the alternate links doesn't exactly match what is redirected to).

Sad: have to change A LOT of PIDs
Happy: Can write something that recursively does all csvs and xml in the repo

Priority: lower than the main pygeoapi PRs that Greg is working on

@dblodgett-usgs
Copy link
Member

dang -- that's my bad. Easy to correct the regex ones. I'll fix the ones I created over in geoconnex_prep.

@ksonda ksonda mentioned this issue Jun 17, 2020
4 tasks
@ksonda
Copy link
Member Author

ksonda commented Jun 29, 2020

Checked in corrections to /ref/ PIDs. Leaving CUAHSI for the remainder of #55

@ksonda ksonda closed this as completed Jun 29, 2020
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

No branches or pull requests

2 participants