Skip to content

Commit 21cc1b2

Browse files
candleindarkclaude
andcommitted
Remove no-op DANDI_ALLOW_LOCALHOST_URLS from test configs
This variable was consumed by the dandi-archive Django server to permit `localhost` URLs in asset `contentUrl` fields during testing (where assets are stored in a local MinIO instance). It was removed from the test `docker-compose.yml` in commit 09c1b75 when dandi-archive switched to `dandiapi.settings.development`, which handles this by default, making the variable a no-op in the test environment. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent fb5e2f0 commit 21cc1b2

2 files changed

Lines changed: 0 additions & 2 deletions

File tree

.github/workflows/run-tests.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ jobs:
2222
runs-on: ${{ matrix.os }}
2323
env:
2424
NO_ET: 1
25-
DANDI_ALLOW_LOCALHOST_URLS: "1"
2625
DANDI_PAGINATION_DISABLE_FALLBACK: "1"
2726
DANDI_TESTS_PERSIST_DOCKER_COMPOSE: "1"
2827
strategy:

tox.ini

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ envlist = lint,typing,py3
33

44
[testenv]
55
setenv =
6-
DANDI_ALLOW_LOCALHOST_URLS=1
76
DANDI_PAGINATION_DISABLE_FALLBACK=1
87
passenv = DANDI_*,USER,DBUS_SESSION_BUS_ADDRESS
98
extras =

0 commit comments

Comments
 (0)