Ticket #3 (closed defect: fixed)

Opened 3 years ago

Last modified 15 months ago

[Core] GET instead of POST in delete action (Forum)

Reported by: admin Owned by: MC-arjen
Priority: minor Milestone: MediaMosa 2.2
Component: Core Version: 1.7.0
Keywords: Cc:
MoSCoW: Should Have Estimated time after impact analysis:
Related to project: none Tested: yes
Accepted: yes Estimated Hours:

Description

Hoofdstuk 9.21 beschrijft het verwijderen van favorieten, doormiddel van GET request

[GET] /user/{fav_user_id}/favorites/delete

De POST request werkt ook, maar volgens mij moet het een van beide zijn en wel POST en niet GET. Elders zijn de 'delete' acties ook POSTs. Als je iets aan de toestand wijzigt (CUD) gebruik je toch POST, als je alleen leest (R) GET?

Change History

Changed 3 years ago by admin

Additional information can be found: http://mantis.kennisnet.nl/view.php?id=16809

Changed 3 years ago by erik

  • owner set to MC-arjen
  • status changed from new to assigned

Fix in SVN

Changed 2 years ago by robert

  • status changed from assigned to closed
  • resolution set to fixed
  • component set to Core
  • related_to set to none

Changed 2 years ago by Michiel.Schok

  • accepted changed from no to yes

SURFmedia gebruikt nu [post], dus dit is in orde.
[get] mag wat ons betreft anytime verdwijnen, ook in 1.7.1 als dat eenvoudiger is.

Changed 2 years ago by Frans

  • version set to 1.7.0

Changed 2 years ago by Frans

  • accepted changed from yes to no

So what happend here? 4 months ago both GET and POST methods worked already. Situation is unchanged. Would like to see GET method depreciated.

Changed 2 years ago by Michiel.Schok

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 2 years ago by forgacs

  • status changed from reopened to assigned

Changed 2 years ago by Frans

  • milestone changed from MediaMosa 1.7 to MediaMosa 2.1

Moved to MediaMosa 2.1

Changed 2 years ago by Frans

  • summary changed from 0016809: 1 [Core] GET instead of POST in delete action (Forum) to [Core] GET instead of POST in delete action (Forum)

Changed 2 years ago by Frans

  • moscow set to Should Have

Changed 20 months ago by Frans

  • milestone changed from MediaMosa 2.1 to MediaMosa 2.2

In MM 2.1. both GET and POST will work here. Issue still valid.

Changed 20 months ago by Frans

This calls for a support of 'Depricated' restcalls. An EGA related setting.

Changed 16 months ago by robert

  • status changed from assigned to closed
  • resolution set to fixed

The GET version of this REST call has been made deprecated in v2.3

Changed 15 months ago by Frans

  • tested set to yes
  • accepted changed from no to yes
Note: See TracTickets for help on using tickets.