Skip to content
Snippets Groups Projects
Commit 710cfc7f authored by Samuel GAIST's avatar Samuel GAIST
Browse files

[api_urls] Remove routes giving access to object without beeing fully qualified

These endpoints usually gives access to the "latest" object version
which is not clear.

If this is really something we want to provide it should be using
an explicit route ending for example with "latest" at the end.
parent 63f9c28a
No related branches found
No related tags found
1 merge request!327Refactor update creation api
...@@ -53,19 +53,9 @@ urlpatterns = [ ...@@ -53,19 +53,9 @@ urlpatterns = [
api.RetrieveUpdateDestroyAlgorithmsView.as_view(), api.RetrieveUpdateDestroyAlgorithmsView.as_view(),
name="object", name="object",
), ),
url(
r"^(?P<author_name>\w+)/(?P<object_name>[-\w]+)/$",
api.RetrieveUpdateDestroyAlgorithmsView.as_view(),
name="object",
),
url( url(
r"^(?P<author_name>\w+)/(?P<object_name>[-\w]+)/(?P<version>\d+)/binary/$", r"^(?P<author_name>\w+)/(?P<object_name>[-\w]+)/(?P<version>\d+)/binary/$",
api.binary, api.binary,
name="binary", name="binary",
), ),
url(
r"^(?P<author_name>\w+)/(?P<object_name>[-\w]+)/binary/$",
api.binary,
name="binary",
),
] ]
...@@ -38,9 +38,4 @@ urlpatterns = [ ...@@ -38,9 +38,4 @@ urlpatterns = [
api.RetrieveDatabaseView.as_view(), api.RetrieveDatabaseView.as_view(),
name="object", name="object",
), ),
url(
r"^(?P<database_name>[-\w]+)/$",
api.RetrieveDatabaseView.as_view(),
name="object",
),
] ]
...@@ -53,9 +53,4 @@ urlpatterns = [ ...@@ -53,9 +53,4 @@ urlpatterns = [
api.RetrieveUpdateDestroyDataFormatsView.as_view(), api.RetrieveUpdateDestroyDataFormatsView.as_view(),
name="object", name="object",
), ),
url(
r"^(?P<author_name>\w+)/(?P<object_name>[a-zA-Z0-9_\-]+)/$",
api.RetrieveUpdateDestroyDataFormatsView.as_view(),
name="object",
),
] ]
...@@ -38,48 +38,24 @@ urlpatterns = [ ...@@ -38,48 +38,24 @@ urlpatterns = [
api.ShareExperimentView.as_view(), api.ShareExperimentView.as_view(),
name="share", name="share",
), ),
url(
r"^(?P<author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/share/$",
api.ShareExperimentView.as_view(),
{"toolchain_author_name": None},
name="share",
),
# Start # Start
url( url(
r"^(?P<author_name>\w+)/(?P<toolchain_author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/start/$", r"^(?P<author_name>\w+)/(?P<toolchain_author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/start/$",
api.StartExperimentView.as_view(), api.StartExperimentView.as_view(),
name="start", name="start",
), ),
url(
r"^(?P<author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/start/$",
api.StartExperimentView.as_view(),
{"toolchain_author_name": None},
name="start",
),
# Cancelling # Cancelling
url( url(
r"^(?P<author_name>\w+)/(?P<toolchain_author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/cancel/$", r"^(?P<author_name>\w+)/(?P<toolchain_author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/cancel/$",
api.CancelExperimentView.as_view(), api.CancelExperimentView.as_view(),
name="cancel", name="cancel",
), ),
url(
r"^(?P<author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/cancel/$",
api.CancelExperimentView.as_view(),
{"toolchain_author_name": None},
name="cancel",
),
# Reseting # Reseting
url( url(
r"^(?P<author_name>\w+)/(?P<toolchain_author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/reset/$", r"^(?P<author_name>\w+)/(?P<toolchain_author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/reset/$",
api.ResetExperimentView.as_view(), api.ResetExperimentView.as_view(),
name="reset", name="reset",
), ),
url(
r"^(?P<author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/reset/$",
api.ResetExperimentView.as_view(),
{"toolchain_author_name": None},
name="reset",
),
# Attestations # Attestations
url( url(
r"^(?P<attestation_number>\d+)/", r"^(?P<attestation_number>\d+)/",
...@@ -97,10 +73,4 @@ urlpatterns = [ ...@@ -97,10 +73,4 @@ urlpatterns = [
api.RetrieveUpdateDestroyExperimentView.as_view(), api.RetrieveUpdateDestroyExperimentView.as_view(),
name="object", name="object",
), ),
url(
r"^(?P<author_name>\w+)/(?P<toolchain_name>[-\w]+)/(?P<version>\d+)/(?P<name>[-\w]+)/$",
api.RetrieveUpdateDestroyExperimentView.as_view(),
{"toolchain_author_name": None},
name="object",
),
] ]
...@@ -53,9 +53,4 @@ urlpatterns = [ ...@@ -53,9 +53,4 @@ urlpatterns = [
api.RetrieveUpdateDestroyLibrariesView.as_view(), api.RetrieveUpdateDestroyLibrariesView.as_view(),
name="object", name="object",
), ),
url(
r"^(?P<author_name>\w+)/(?P<object_name>[-\w]+)/$",
api.RetrieveUpdateDestroyLibrariesView.as_view(),
name="object",
),
] ]
...@@ -48,9 +48,4 @@ urlpatterns = [ ...@@ -48,9 +48,4 @@ urlpatterns = [
api.RetrieveUpdateDestroyToolchainsView.as_view(), api.RetrieveUpdateDestroyToolchainsView.as_view(),
name="object", name="object",
), ),
url(
r"^(?P<author_name>\w+)/(?P<object_name>[-\w]+)/$",
api.RetrieveUpdateDestroyToolchainsView.as_view(),
name="object",
),
] ]
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment