Hi Alessandro,

if `allowMethod` is simply going to be removed, QGIS-documentation has to be 
updated. I've created the issue https://github.com/qgis/QGIS/issues/36163

If it's going to be removed I could probably create an PR for QGIS and 
QGIS-documetation :)

Dražen



‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Saturday, May 2, 2020 10:17 PM, Alessandro Pasotti <[email protected]> 
wrote:

> On Sat, May 2, 2020 at 3:27 PM Dražen Odobašić
> [email protected] wrote:
>
> > Hi,
> > I've been toying around with QGIS Server Python plugins and it seems like 
> > `allowMethod` in QgsService does not get executed. A simple `git grep 
> > allowMethod` on the master branch shows few definitions but not sure if 
> > it's actually called from anywhere (maybe some external lib does actually 
> > invoke it, don't know)
> > Is this method actually used and how does it get called? Maybe it does work 
> > in C++, but not in Python?
>
> Hi Dražen,
>
> I think that you actually spotted an API leftover. I don't think that
> that method is needed, the best place for all checks is probably
> executeRequest where the service/API can decide if a request is valid
> according to its internal logic.
>
> But of course if we want to keep it we should at least check for the
> HTTP method before we accept the request.
>
> Would you please file a ticket?.
>
>
> ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Alessandro Pasotti
> QCooperative: www.qcooperative.net
> ItOpen: www.itopen.it


_______________________________________________
QGIS-Developer mailing list
[email protected]
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Reply via email to