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

Trailing slash may encounter issue when creating GSManager #51

Closed
bmkor opened this issue Feb 19, 2022 · 4 comments
Closed

Trailing slash may encounter issue when creating GSManager #51

bmkor opened this issue Feb 19, 2022 · 4 comments
Assignees
Milestone

Comments

@bmkor
Copy link

bmkor commented Feb 19, 2022

First of all, big thanks for this wonderful R library.

Some details on the versions:
Geoserver vesion 2.17.0
R version 4.0.2
geosapi version 0.5-1

I encountered error (http status 500) when creating GSManager through running GSManager$new. Upon looking at the code, found that this line would add a trailing slash, i.e. https://[GEOSERVER HOST]/geoserver/rest/ when creating the connection and this would return error in my case. Managed to solve this issue by hacking through removing the trailing slash via nginx rewriting rule.

I am not certain if this is only my geoserver setting and would like to share this out for your information.

Thanks a lot.

@mathieu-massaviol
Copy link

Hi,

I have the same issue with my configuration :

Geoserver vesion 2.20.0
R version 4.2.0
geosapi version 0.6

Thanks in advance for the help !

@eblondel eblondel self-assigned this May 30, 2022
@eblondel eblondel added this to the 0.6 milestone May 30, 2022
eblondel added a commit that referenced this issue May 30, 2022
@eblondel
Copy link
Owner

@bmkor @mathieu-massaviol I've applied a fix for this. Can you test it?

eblondel added a commit that referenced this issue May 31, 2022
@mathieu-massaviol
Copy link

It works perfectly now, thanks @eblondel !

@eblondel
Copy link
Owner

eblondel commented Jun 1, 2022

part of geosapi 0.6-1, just published on CRAN

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants