Upgrade Go version from 1.24 to 1.26 by karwaankit32 · Pull Request #4737 · prebid/prebid-server · GitHub
Skip to content

Upgrade Go version from 1.24 to 1.26#4737

Merged
bsardo merged 15 commits into
prebid:masterfrom
karwaankit32:go-version-upgrade
Jun 23, 2026
Merged

Upgrade Go version from 1.24 to 1.26#4737
bsardo merged 15 commits into
prebid:masterfrom
karwaankit32:go-version-upgrade

Conversation

@karwaankit32

@karwaankit32 karwaankit32 commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

Upgrading Go version build from 1.24 and 1.26.
Doing version compatibility fixes

  • Fixed Go 1.26 compatibility in timeout handling by recognizing wrapped context deadline exceeded errors
  • Cleaned up formatting-related code by removing unnecessary fmt.Sprintf usage where a static string was sufficient. If not done, it fails during go vet and tests also fail

Testing
Ran all Unit tests
Tested following endpoints -

  • /cookie_sync
  • /openrtb/auction

@karwaankit32 karwaankit32 marked this pull request as draft April 1, 2026 05:52
@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Apr 1, 2026

Copy link
Copy Markdown

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 068c6d3

adapterstest

Refer here for heat map coverage report

github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:23:	BidOnTags			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:33:	SampleBid			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:46:	VerifyStringValue		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:53:	VerifyIntValue			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:60:	VerifyBoolValue			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:67:	VerifyBannerSize		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:70:		RunJSONBidderTest		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:83:		RunSingleJSONBidderTest		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:105:		loadFile			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:127:		runSpec				0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:135:		getTestExtraRequestInfo		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:185:		expectsErrors			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:194:		ToRequestData			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:215:		ToResponseData			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:244:		assertMakeRequestsOutput	0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:279:		assertErrorList			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:307:		assertMakeBidsOutput		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:336:		diffHttpRequests		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:371:		diffBids			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:386:		diffOrtbBids			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:399:		diffBidVideo			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:410:		diffJson			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:448:		testMakeRequestsImpl		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:469:		getDataRaceTestCopies		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:490:		testMakeBidsImpl		0.0%
total:											(statements)			0.0%

@karwaankit32 karwaankit32 marked this pull request as ready for review April 1, 2026 06:31
@karwaankit32 karwaankit32 requested a review from SyntaxNode April 1, 2026 06:31
@SyntaxNode

Copy link
Copy Markdown
Contributor

Curious, entire json and yaml files are showing up as a change. Did you inadvertently change the line endings?

Comment thread .github/workflows/adapter-code-coverage.yml Outdated
Comment thread .github/workflows/validate-merge.yml Outdated
Comment thread .github/workflows/validate.yml Outdated
Comment thread adapters/adapterstest/adapter_test_util.go
Comment thread endpoints/openrtb2/amp_auction_test.go
Comment thread exchange/bidder_test.go Outdated
Comment thread modules/fiftyonedegrees/devicedetection/hook_raw_auction_request.go
Comment thread Dockerfile Outdated
Comment thread go.mod Outdated
Comment thread README.md Outdated
@linux019

linux019 commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

The go version should be at least 1.26.2. Previous versions 1.26.x have bugs in delve debugger

@bsardo bsardo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I understand Go 1.26 has a stricter linter which is driving some of these changes and we will accept those changes as part of this PR. However, typically those changes should be in a separate PR that would be merged prior to the Go version upgrade so please follow that guidance going forward.

Comment thread .github/workflows/validate.yml Outdated
Comment thread README.md Outdated
@bsardo

bsardo commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Also note that this PR will be its own release.

@anishb-ms

Copy link
Copy Markdown
Contributor

Curious, entire json and yaml files are showing up as a change. Did you inadvertently change the line endings?

Good catch — yes, the editor saved those four files with CRLF line endings, which made the entire file render as changed even though only a few lines actually differ. Fixed it

@bsardo

bsardo commented May 14, 2026

Copy link
Copy Markdown
Collaborator

@anishb-ms now that we merged in my change to the adapter code coverage workflow, can you merge with master to verify that that workflow failure goes away here?

@github-actions

Copy link
Copy Markdown

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 454a8b2

adapterstest

Refer here for heat map coverage report

github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:23:	BidOnTags			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:33:	SampleBid			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:46:	VerifyStringValue		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:53:	VerifyIntValue			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:60:	VerifyBoolValue			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/adapter_test_util.go:67:	VerifyBannerSize		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:70:		RunJSONBidderTest		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:83:		RunSingleJSONBidderTest		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:105:		loadFile			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:127:		runSpec				0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:135:		getTestExtraRequestInfo		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:185:		expectsErrors			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:194:		ToRequestData			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:215:		ToResponseData			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:244:		assertMakeRequestsOutput	0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:279:		assertErrorList			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:307:		assertMakeBidsOutput		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:336:		diffHttpRequests		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:371:		diffBids			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:386:		diffOrtbBids			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:399:		diffBidVideo			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:410:		diffJson			0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:448:		testMakeRequestsImpl		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:469:		getDataRaceTestCopies		0.0%
github.com/prebid/prebid-server/v4/adapters/adapterstest/test_json.go:490:		testMakeBidsImpl		0.0%
total:											(statements)			0.0%

@anishb-ms

Copy link
Copy Markdown
Contributor

@anishb-ms now that we merged in my change to the adapter code coverage workflow, can you merge with master to verify that that workflow failure goes away here?

All validations are passing successfully now.

@github-actions

Copy link
Copy Markdown

@bsardo bsardo changed the title Upgrading Go version from 1.24 to 1.26 Upgrade Go version from 1.24 to 1.26 Jun 23, 2026
@bsardo bsardo merged commit 0ef82cb into prebid:master Jun 23, 2026
8 checks passed
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.

6 participants