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

Change WMS/WFS/WCS/WMTS settings #157

Closed
pesekon2 opened this issue Jul 19, 2024 · 3 comments · Fixed by #166
Closed

Change WMS/WFS/WCS/WMTS settings #157

pesekon2 opened this issue Jul 19, 2024 · 3 comments · Fixed by #166
Labels
enhancement New feature or request

Comments

@pesekon2
Copy link
Contributor

Changing service settings (such as timeout) through a function would be a great step towards automation without a need to change this manually after all the imports, etc.

@iboates
Copy link
Collaborator

iboates commented Jul 20, 2024

I think this functionality should be included in this PR: #153

It hasn't been published to pypi yet. You could check to see if it works for you by installing it like this:

pip install git+https://github.com/gicait/geoserver-rest@master

You could test it out. When you make the Geoserver object, pass a dictionary to the request_params parameter, they should be included in every subsequent request to the geoserver instance.

@iamtekson
Copy link
Collaborator

I just released the new version v2.8.2. I think it will solve the issue. If the issue still exist, please feel free to open this issue.

@pesekon2
Copy link
Contributor Author

@iamtekson @iboates: As far as I know (and was able to reach with the code), the timeout in request_params changes the timeout for requests like get or push (i.e., in terms of this package, it would be things like get_workspaces()). But it does not change it for services like WMS. At least when I set it to whatever value, it limits the named ones while the WMS still has the default timeout of 60s (which I would liike to change and have to do it manually now).

I believe the parameter of the WMS I should set is maxRenderingTime, according to the docs
. But if I am not mistaken, it is not possible to set this parameter the recommended way.

Naturally, if I am mistaken, I would be happy to hear that and find the way to proceed. :-)

PS: Unfortunately, I do not have rights to reopen the issue.

@iamtekson iamtekson reopened this Jul 28, 2024
@iamtekson iamtekson added the enhancement New feature or request label Jul 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants