search

icatproject / ids.server

The server component of the ICAT Data Service
Other
3 stars 3 forks source link

Consider to remove POST implementation of put #162

Open RKrahl opened 1 month ago

RKrahl commented 1 month ago

Just noticed the following in the documentation:

This is an alternative to using PUT on the put resource. All the same arguments appear as form fields. In addition there are two boolean fields wrap and padding which should be set to true as a CORS work around. These two fields will be removed shortly as they are only required by the old (GWT based) topcat.

It is years now that the old (GWT based) topcat is not in use any more, but these fields are still in the code. :astonished:

Check whether anybody still needs that and remove it if not.

RKrahl commented 1 month ago

As an alternative we might consider to remove the POST implementation of the put call altogether. Need to be discussed in the collaboration.

RKrahl commented 1 month ago

Discussed in today's collaboration meeting: we rather consider to remove the POST implementation of the put call altogether. I will announce that again to the mailing list and if no one objects, we will deprecate that call now and remove it in 3.0.0.