Update dependencies #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

carabasdaniel merged 22 commits into main from updates

Apr 30, 2025

Contributor

carabasdaniel commented Apr 10, 2025

No description provided.

carabasdaniel added 6 commits

April 10, 2025 18:54


          Update dependencies

77e6c5e


          Fix sdk linting issues

7b179a1


          Fix linting for openapi, okta and ldap plugins

bd76d0e


          Fix linting in jumpcloud, google and fusionauth plugins

fbfa2de


          Fix linting for azuread, azuredb2c and cognito plugins


          Fix auth0 and cli linting errors

27587c4

carabasdaniel force-pushed the updates branch from 96d736e to 8d93d47 Compare

April 14, 2025 16:21


          Update ci go linter and go releaser versions

8ba9f1c

carabasdaniel force-pushed the updates branch from 8d93d47 to 8ba9f1c Compare

April 14, 2025 16:29

carabasdaniel requested a review from ronenh

April 14, 2025 16:29

ronenh reviewed

View reviewed changes

.golangci.yaml Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

.golangci.yaml Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

cli/cmd/ds-load/main.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

cli/pkg/app/cli.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

cli/pkg/app/exec.go Show resolved Hide resolved

ronenh reviewed

View reviewed changes

makefile Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

makefile Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/app/cli.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/azureclient/credential.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/azureclient/credential.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azureadb2c/pkg/azureclient/credential.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/cognito/pkg/fetch/fetch.go Outdated Show resolved Hide resolved


          Use linting rules from topaz and update formatting with gofumpt

11440b6

carabasdaniel force-pushed the updates branch from bc45c28 to 760aac7 Compare

April 15, 2025 09:22


          Update error messages and address review comments

fdba13a

carabasdaniel force-pushed the updates branch from 760aac7 to fdba13a Compare

April 15, 2025 09:24

carabasdaniel requested review from florindragos and ronenh

April 23, 2025 13:39

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go

    
              		return nil, err

              	}

              	obj["email_verified"] = user.GetEmailVerified()

Contributor

ronenh Apr 23, 2025

Why does "email_verified" need to be added explicitly? Wouldn't it be part of the map unmarshaled above (in line 131)?

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/auth0/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/azuread/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/jumpcloud/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

plugins/jumpcloud/pkg/fetch/fetch.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

sdk/common/version/version.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

sdk/fetcher/fetcher.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

sdk/fetcher/fetcher.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

sdk/transform/functions.go Show resolved Hide resolved


          Address review comments

69c5807

ronenh reviewed

View reviewed changes

sdk/fetcher/fetcher.go Outdated Show resolved Hide resolved

carabasdaniel added 2 commits

April 24, 2025 19:00


          Fix generic fetcher map yield helper

0170d18


          Create a custom error writer in sdk

ebcb245

carabasdaniel requested a review from ronenh

April 25, 2025 09:31

ronenh reviewed

View reviewed changes

sdk/common/error_writer.go Outdated Show resolved Hide resolved

ronenh reviewed

View reviewed changes

sdk/common/error_writer.go Outdated Show resolved Hide resolved


          Update error writer

ea1a067

carabasdaniel force-pushed the updates branch from 4aa5d3a to ea1a067 Compare

April 29, 2025 08:42

carabasdaniel requested a review from ronenh

April 29, 2025 08:43


          ErrorWriter embeds an io.Writer

670337f

ronenh approved these changes

View reviewed changes

carabasdaniel merged commit f1eb2f6 into main

14 checks passed

carabasdaniel deleted the updates branch

April 30, 2025 14:51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet