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

WFS should use POST when getting UF/parcelle info #420

Closed
landryb opened this issue Oct 24, 2018 · 5 comments
Closed

WFS should use POST when getting UF/parcelle info #420

landryb opened this issue Oct 24, 2018 · 5 comments
Assignees
Milestone

Comments

@landryb
Copy link
Member

landryb commented Oct 24, 2018

When clicking on 'Fiche unité foncière' on a given parcelle, i sometimes get 400 codes from the webserver, because cadastrapp does a GET on the wfs via a GetFeature, passing the parcelle polygon encoded in the URL, and thus the request headers are 8k and hit default server limits.

Sure, i can bump limits here and there in the various proxies in the chain between the client and the webapp, but in that case it would be much simpler to use POST, which is made for this usecase....

@landryb
Copy link
Member Author

landryb commented Oct 24, 2018

Even after bumping the buffer sizes in the nginx proxies (ie set large_client_header_buffers 4 16k;), it still fails, and it seems it also hits a limit in the s-p tomcat, which logs:

24-Oct-2018 11:12:31.700 INFOS [http-nio-8180-exec-1] org.apache.coyote.http11.Http11Processor.service Error parsing HTTP request header
 Note: further occurrences of HTTP header parsing errors will be logged at DEBUG level.
 java.lang.IllegalArgumentException: L'entête de requête est trop important

@MaelREBOUX
Copy link
Member

Tiens ? Je croyais qu'on avait tout passé en POST il y a qqs temps déjà. Ma mémoire failli. @pierrejego ?

@pierrejego
Copy link
Member

Non on avait pas tout passé en POST. Je modifie les services qui évolue pour les mettre en POST au fur et à mesure. Il en reste encore en GET.

@pierrejego pierrejego self-assigned this Dec 14, 2018
@pierrejego pierrejego added this to the v 1.8 milestone Dec 14, 2018
@pierrejego pierrejego assigned pierrejego and unassigned pierrejego Mar 26, 2019
@Gaetanbrl
Copy link
Collaborator

Passage GET vers POST pour cette requête GetFeature.

@landryb
Copy link
Member Author

landryb commented Apr 16, 2019

Nice !

pierrejego pushed a commit to jdev-org/cadastrapp that referenced this issue Apr 29, 2019
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

4 participants