Skip to content

Fix gin request ID - #19

Closed
rodigoalr wants to merge 1 commit into
masterfrom
fix/fix_gin_request
Closed

rodigoalr wants to merge 1 commit into
masterfrom
fix/fix_gin_request

Conversation

@rodigoalr

Copy link
Copy Markdown
Contributor
../../pkg/mod/github.com/atarantini/ginrequestid@v0.0.0-20210724204029-300df7d153f5/requestid.go:22:13: assignment mismatch: 1 variable but uuid.NewV4 returns 2 values

El fix es muy simple, ya tiene un PR atarantini/ginrequestid#3 que no se ha aceptado desde hace tiempo. Al ser una librería muy chica(una función), que parece que no tiene mantenimiento alguno, se plantea en traer ese código para acá y hacer el fix de nuestro lado.

@github-actions

Copy link
Copy Markdown

New Coverage 72.2% of statements
Patch Coverage 46.2% of changed statements (6/13)

Coverage provided by https://github.com/seriousben/go-patch-cover-action

@nicodelara

Copy link
Copy Markdown
Contributor

@rodigoalr lo veo bien. Si están trabados con este tema pueden downgradear a otra versión que si es compatible.
Ejecutando un go get github.com/atarantini/ginrequestid@6d9eee666701 debería salir andando solo.

@rodigoalr rodigoalr closed this Sep 30, 2022
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