actualbudget / actualbudget/actual

[Bug]: No way to set server_hostname when setting up OIDC with the UI

Abierto
#5,836 2 comentarios 0 reacciones 0 asignados Ver en GitHub
openid
Lenguaje dominante
TypeScript
Estrellas
28.7k
Forks
3k
Merge medio
2 d 11 h
PR fusionados (30 d)
65

Descripción

### Verified issue does not already exist?

- [x] I have searched and found no existing issue

### What happened?

I configured the DNS name `actual.example.com` after setting the app up initially using forwarded ports and the `localhost` DNS name.

While doing the initial setup with the `localhost` DNS name, I used the UI to configure OIDC.

This caused actual to save `"server_hostname":"http://localhost:5006"` in the OIDC settings in the database.

When I got my `actual.example.com` DNS name set up, actual continued trying to use `http://localhost:5006` as the redirect base URL.

I tried setting the `ACTUAL_OPENID_SERVER_HOSTNAME` variable to `actual.example.com`, but it didn't change the value in the database. (Maybe because I omitted `https://`?)

I ended up updating the value in the database manually, and it's now working.

### How can we reproduce the issue?

1. configure a fresh install of actual server using the DNS name "localhost".
2. Add OIDC settings via the UI, while still using localhost.
3. Connect with the DNS name that fits the redirect configured in OIDC provider. Logging in should give a redirect error.

### Where are you hosting Actual?

Docker

### What browsers are you seeing the problem on?

Firefox

### Operating System

Linux

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.