Skip to content

Conversation

@cshagen
Copy link
Collaborator

@cshagen cshagen commented Feb 2, 2025

  • Korrektur mittlerer Graph
  • Korrektur Zeit-Format

Comment on lines 10 to 12
server: {
allowedHosts: true,
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Abgesehen davon, dass die Einrückung nicht passt: warum ist diese Einstellung erforderlich? Ich sehe keine Grund.
Laut Doku wird dringend davon abgeraten. https://vite.dev/config/server-options#server-allowedhosts

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohne das bekomme ich mit der aktuellen Vita-Version diese Fehlermeldung im Browser, wenn das Theme lokal starte:

Blocked request. This host ("openwb-dev") is not allowed.
To allow this host, add "openwb-dev" to `server.allowedHosts` in vite.config.js.

Es hilft aber nicht, wenn ich meinen internen Hostnamen auf Github poste, ausserdem kann der ja auch mal ändern. Die Config sollte ja auch nur dann greifen, wenn das Colors-Theme lokal mit "npm run dev" gestartet wird. Es sollte daher für den normalen Betrieb keinen Sicherheitsabstand-Impact geben.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Zugriff per IP reicht Dir nicht?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Danke für den Tipp. IP-Adresse funktioniert. Ich habe die server.allowedHosts-Config wieder rausgenommen.

@benderl benderl merged commit bdee5d1 into openWB:master Feb 4, 2025
1 check passed
@cshagen cshagen deleted the web-fixes branch February 9, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants