Server Error when using: /api/internal/extensions-awaiting-review/ #263

New Issue

Dalai Felinto · 2024-10-14T12:59:48+02:00

Dalai Felinto commented

2024-10-14 12:59:48 +02:00

Visit: https://extensions.blender.org/api/internal/extensions-awaiting-review/

It returns a 500 Server Error.

The API code is assuming all the extensions have a valid latest_version. However this is not the case to some extensions.

I don't know:

Which extensions (I have trouble accessing the production database since we moved to the new servers).
Whether this is a valid assumption (and the fix is on the data-base).
If this is a invalid assumption and the code needs to reflect that (this would make for an easy fix).

For the records, the issue is on the function extensions_awaiting_review, on the line:
version.extension = extension, because version is None.

1. Visit: https://extensions.blender.org/api/internal/extensions-awaiting-review/ It returns a 500 Server Error. The API code is assuming all the extensions have a valid latest_version. However this is not the case to some extensions. I don't know: * Which extensions (I have trouble accessing the production database since we moved to the new servers). * Whether this is a valid assumption (and the fix is on the data-base). * If this is a invalid assumption and the code needs to reflect that (this would make for an easy fix). For the records, the issue is on the function `extensions_awaiting_review`, on the line: `version.extension = extension`, because version is None.

Dalai Felinto added the

labels 2024-10-14 12:59:49 +02:00

Dalai Felinto commented

2024-10-14 13:09:26 +02:00

Found it, the issue is the :
https://extensions.blender.org/approval-queue/render-at-specific-dpi/

In this case the author deleted the version (probably intended to delete the extension?).

Which means indeed we should account for this scenario.

Found it, the issue is the : https://extensions.blender.org/approval-queue/render-at-specific-dpi/ In this case the author deleted the version (probably intended to delete the extension?). Which means indeed we should account for this scenario.

Dalai Felinto closed this issue

2024-10-14 13:09:27 +02:00

Nika Kutsniashvili commented

2024-10-14 14:31:26 +02:00

Interesting...

Which means indeed we should account for this scenario.

@dfelinto can you delete that extension in the meantime? Will that let me pull from server?

Interesting... > Which means indeed we should account for this scenario. @dfelinto can you delete that extension in the meantime? Will that let me pull from server?

Dalai Felinto referenced this issue from a commit

2024-10-14 14:34:23 +02:00

Fix #263: Server error on extensions-awaiting-review

Dalai Felinto commented

2024-10-14 14:37:18 +02:00

@nickberckley no need, I pushed a fix (and deployed in production).

❤️ 1

Sign in to join this conversation.