Skip to content

Create login audit table#56

Merged
suvikankare merged 2 commits intomainfrom
user-table
Dec 5, 2025
Merged

Create login audit table#56
suvikankare merged 2 commits intomainfrom
user-table

Conversation

@suvikankare
Copy link
Member

@suvikankare suvikankare commented Dec 4, 2025

This change is Reviewable

@suvikankare suvikankare force-pushed the user-table branch 2 times, most recently from dbae80a to 6008b3b Compare December 4, 2025 10:53
@suvikankare suvikankare marked this pull request as ready for review December 4, 2025 12:06
@suvikankare suvikankare force-pushed the user-table branch 2 times, most recently from 9ebb327 to b96c98e Compare December 4, 2025 13:32
Copy link
Contributor

@Huulivoide Huulivoide left a comment

Choose a reason for hiding this comment

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

@Huulivoide reviewed 7 of 10 files at r1, 8 of 8 files at r2, all commit messages.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on @suvikankare)


src/main/resources/db/migration/V1__create_login_audit_table.sql line 12 at r2 (raw file):

CREATE INDEX IF NOT EXISTS idx_login_audit_timestamp ON login_audit(login_timestamp);

-- Grant permissions to the database user

Hasura käyttäjällekin vois tässä saman tien antaa lukuoikat. Viimeistään seuraavan tiketin osana tulee sit jos ei tässä jo. Vältytään toiselta PR:ltä tähän repoon


src/main/kotlin/fi/hsl/jore4/auth/oidc/OIDCCodeExchangeService.kt line 100 at r2 (raw file):

            loginAuditService?.let {
                val userId = idToken.jwtClaimsSet.subject
                val userName = idToken.jwtClaimsSet.getStringClaim("name")

Kivahan se ois jos tän näin helposti sais sieltä ulos, mut onks tää tarkastettu että siellä on "name" kenttä tuolla Entra ID:n tokenissa? Vai pitääkö tässä kiepauttaa ja kyellä userInfo API:n kautta lisätiedot.


pom.xml line 30 at r2 (raw file):

        <versions.maven.enforcer>3.5.0</versions.maven.enforcer>
        <versions.maven.ktlint>3.5.0</versions.maven.ktlint>
        <versions.maven.flyway>11.17.0</versions.maven.flyway>

Authi palvelussa on toistaseks käytössä Spring Boot versio 3.3 ja sen mukana tulee depsut: https://docs.spring.io/spring-boot/3.3/appendix/dependency-versions/coordinates.html

Eli siellä on käytössä V10 sarjan flyway. Tässä että maven puolella on uudempi versio plugarista köytössä ei varmaan oo haittaa, mutta ehkä tänkin vosi laittaa nyt ens alkuun oleen synkassa itse äppiksen sisällä pyörivän version kanssa.

Copy link
Member Author

@suvikankare suvikankare left a comment

Choose a reason for hiding this comment

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

Reviewable status: 9 of 12 files reviewed, 3 unresolved discussions (waiting on @Huulivoide)


pom.xml line 30 at r2 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Authi palvelussa on toistaseks käytössä Spring Boot versio 3.3 ja sen mukana tulee depsut: https://docs.spring.io/spring-boot/3.3/appendix/dependency-versions/coordinates.html

Eli siellä on käytössä V10 sarjan flyway. Tässä että maven puolella on uudempi versio plugarista köytössä ei varmaan oo haittaa, mutta ehkä tänkin vosi laittaa nyt ens alkuun oleen synkassa itse äppiksen sisällä pyörivän version kanssa.

Done.


src/main/kotlin/fi/hsl/jore4/auth/oidc/OIDCCodeExchangeService.kt line 100 at r2 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Kivahan se ois jos tän näin helposti sais sieltä ulos, mut onks tää tarkastettu että siellä on "name" kenttä tuolla Entra ID:n tokenissa? Vai pitääkö tässä kiepauttaa ja kyellä userInfo API:n kautta lisätiedot.

Joo olin lukenu huonosti, done


src/main/resources/db/migration/V1__create_login_audit_table.sql line 12 at r2 (raw file):

Previously, Huulivoide (Jesse Jaara) wrote…

Hasura käyttäjällekin vois tässä saman tien antaa lukuoikat. Viimeistään seuraavan tiketin osana tulee sit jos ei tässä jo. Vältytään toiselta PR:ltä tähän repoon

Done.

Copy link
Contributor

@Huulivoide Huulivoide left a comment

Choose a reason for hiding this comment

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

@Huulivoide reviewed 3 of 3 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @suvikankare)

Copy link
Contributor

@Huulivoide Huulivoide left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @suvikankare)

@suvikankare suvikankare merged commit b76b47f into main Dec 5, 2025
19 checks passed
@suvikankare suvikankare deleted the user-table branch December 5, 2025 11:59
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