Skip to content

bouncer 2.0#95

Merged
aateekoohenna merged 11 commits intomainfrom
feature/bouncer2
Mar 20, 2026
Merged

bouncer 2.0#95
aateekoohenna merged 11 commits intomainfrom
feature/bouncer2

Conversation

@aateekoohenna
Copy link
Contributor

@aateekoohenna aateekoohenna commented Mar 18, 2026

Korvataan nykyinen bouncer-palvelu sellaisella, joka ei enää mainosta palvelimen nimeä mDNS:llä vaan nojaa vanhaan kunnon DNS:ään.

Loput on työn alla #94:ssä; tämä PR sisältää pieniä muutoksia dnsmasq-templateen ja examnet-skriptiin, jotta tämä olisi tällaisenaan itsenäinen. Tästä syystä uutta bounceriakaan ei käynnistetä suoraan, vaan entinen digabi2-examnet-bouncer-skripti pienin muutoksin on vielä välissä.

Toistaiseksi uuden palvelun nimi on bouncer2, mutta vanha nimi voidaan palauttaa sitten kun se on ajankohtaista.

for the time being, the temporal creature
- set dhcp search domain in dnsmasq scaffolding
- expect redirect paths to have trailing slashes
@aateekoohenna aateekoohenna marked this pull request as ready for review March 19, 2026 09:58
@mtreinik
Copy link
Contributor

Kaikkien API-kutsujen testeissä voisi tarkistaa, että responsejen statuskoodit ovat haluttuja

@aateekoohenna
Copy link
Contributor Author

Ei niitä haluttuja koodeja oo kyl määritelty sen tarkemmin, ainoastaan 307 on noissa uudelleenohjauksissa

if your name isn't ytl-linux-digabi2-examnet --daemon, you don't
need to touch it anyway
readonly BIN_XARGS=/usr/bin/xargs
readonly BIN_OPENSSL=/usr/bin/openssl
readonly BIN_DIGABI2_EXAMNET_BOUNCER=/usr/local/sbin/digabi2-examnet-bouncer
readonly BIN_DIGABI2_EXAMNET_BOUNCER=/usr/local/libexec/ytl-linux-digabi2-examnet/bouncer
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Siirsin tän libexecin alle hygienian vuoksi

@aateekoohenna aateekoohenna merged commit 645fa5c into main Mar 20, 2026
1 check passed
@aateekoohenna aateekoohenna deleted the feature/bouncer2 branch March 20, 2026 13:31
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.

3 participants