api snapshot public_url bugfix
with the following code the URL is more appropriate (it basically mirrors the user's request) before: http://localhost/device/{shortid} after http://localhost:8000/device/{shortid}
This commit is contained in:
parent
b4c4ed2689
commit
5e6546169b
|
@ -82,15 +82,14 @@ def NewSnapshot(request):
|
||||||
if not annotation:
|
if not annotation:
|
||||||
return JsonResponse({'status': 'fail'}, status=200)
|
return JsonResponse({'status': 'fail'}, status=200)
|
||||||
|
|
||||||
url = "{}://{}{}".format(
|
url_args = reverse_lazy("device:details", args=(annotation.value,))
|
||||||
request.scheme,
|
url = request.build_absolute_uri(url_args)
|
||||||
settings.DOMAIN,
|
|
||||||
reverse_lazy("device:details", args=(annotation.value,))
|
|
||||||
)
|
|
||||||
response = {
|
response = {
|
||||||
"status": "success",
|
"status": "success",
|
||||||
"dhid": annotation.value[:6].upper(),
|
"dhid": annotation.value[:6].upper(),
|
||||||
"url": url,
|
"url": url,
|
||||||
|
# TODO replace with public_url when available
|
||||||
"public_url": url
|
"public_url": url
|
||||||
}
|
}
|
||||||
return JsonResponse(response, status=200)
|
return JsonResponse(response, status=200)
|
||||||
|
|
Loading…
Reference in a new issue