Ticket #238 (closed enhancement: fixed)

Opened 3 years ago

Last modified 22 months ago

Parameter name change in /asset/{asset_id}/play

Reported by: Michiel.Schok Owned by: robert
Priority: major Milestone: MediaMosa 3.0
Component: Core Version:
Keywords: Cc:
MoSCoW: Must Have Estimated time after impact analysis:
Related to project: none Tested: no
Accepted: no Estimated Hours:

Description

According to http://mediamosa.org/node/5100 the parameter used for providing the group_name(s) in a play request is called 'group_id'.
On other calls, notably the /asset call, the parameter is called 'aut_group_id'.

As the parameter is used for authorization purposes, it is wise to add an 'aut_group_id' parameter to the play call that duplicates the function of 'group_id'. In a subsequent version 'group_id' could be deprecated or removed.

Change History

Changed 2 years ago by Frans

  • moscow changed from Should Have to Must Have
  • milestone changed from MediaMosa X.X to MediaMosa 3.0

Changed 2 years ago by forgacs

  • owner set to forgacs
  • status changed from new to assigned

Changed 2 years ago by forgacs

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

Parameter name "aut_group_id" added in /asset/{asset_id}/play, and "group_id" is marked as depricated.
Also I have checked the tests and other references and I have changed "group_id" to "aut_group_id" in case of /asset/{asset_id}/play call.

Changed 2 years ago by robert

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 2 years ago by robert

  • owner changed from forgacs to robert
  • status changed from reopened to assigned

Changed 2 years ago by robert

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

reviewed

Changed 22 months ago by forgacs

App_id = 1
Test asset: /asset/vM8WljsiZFjnLQlMfJLvVQmS

Setting up ACL:
POST: /mediafile/i2dUHfKETRkbPcimSe0fZO3L/acl
Parameters: aut_group_id[]=test

Testing with the new parameter:
/asset/z1bjdId3ZFB9UNkHRkqz3pft/play?mediafile_id=i2dUHfKETRkbPcimSe0fZO3L&acl_group_id=test

Changed 22 months ago by Michiel.Schok

Peter, in your test you use the 'acl_group_id' parameter.
I presume that it is 'aut_group_id' ?

Changed 22 months ago by forgacs

Both are working.

Note: See TracTickets for help on using tickets.