Skip to content

go-ldap: fix broken build - #16045

Open
xingyaner wants to merge 1 commit into
google:masterfrom
xingyaner:fix-go-ldap
Open

xingyaner wants to merge 1 commit into
google:masterfrom
xingyaner:fix-go-ldap

Conversation

@xingyaner

Copy link
Copy Markdown
Contributor

The fuzzing build failed because build.sh ran go get from the repository root, where no go.mod file exists. The current Go module is located in the v3 directory.

The existing fuzz targets also referenced the legacy github.com/go-ldap/ldap module and fuzz functions that are no longer present in the current source tree.

Update the build script to build from the v3 module directory, use the github.com/go-ldap/ldap/v3 module path, replace the removed fuzz targets with FuzzGetLDAPError, and disable VCS metadata injection during the build.

@github-actions

Copy link
Copy Markdown

xingyaner is a new contributor to projects/go-ldap. The PR must be approved by known contributors before it can be merged. The past contributors are: 0x34d

This branch has not been deployed

No deployments
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.

1 participant