Skip to content

New Adapter: iqiyi #4605

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
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

New Adapter: iqiyi #4605

wants to merge 3 commits into from

Conversation

@Heihuang
Copy link

@Heihuang Heihuang commented Nov 13, 2025

Adding iqiyi prebid server adapter

github-actions[bot]
github-actions bot reviewed Nov 13, 2025
View reviewed changes
@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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, 9d4c906

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	Builder			80.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:38:	MakeRequests		83.9%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:96:	MakeBids		86.7%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:130:	getMediaTypeForImp	80.0%
total:								(statements)		84.5%

@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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, c36b02c

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	Builder			80.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:38:	MakeRequests		83.9%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:96:	MakeBids		86.7%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:130:	getMediaTypeForImp	80.0%
total:								(statements)		84.5%

@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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, cf8485b

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	pickCurrency		60.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	Builder			80.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:45:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:50:	MakeRequests		83.9%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:108:	MakeBids		87.5%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:143:	getMediaTypeForImp	80.0%
total:								(statements)		82.8%

@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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, a937b11

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	pickCurrency		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	Builder			85.7%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:49:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:54:	MakeRequests		96.8%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:112:	MakeBids		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:147:	getMediaTypeForImp	100.0%
total:								(statements)		97.0%

@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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, 3346579

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	pickCurrency		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	Builder			100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:49:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:54:	MakeRequests		94.3%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:114:	MakeBids		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:149:	getMediaTypeForImp	100.0%
total:								(statements)		97.1%

New Adapter: iqiyi
e254d40 
remove go.mod and go.sum files

add go.mod go.sum

format json file
@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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, e254d40

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	pickCurrency		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	Builder			100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:49:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:54:	MakeRequests		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:114:	MakeBids		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:149:	getMediaTypeForImp	100.0%
total:								(statements)		100.0%

@bsardo bsardo added the adapter label Nov 18, 2025
@github-actions
Copy link

github-actions bot commented Nov 19, 2025

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, 9e70614

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	pickCurrency		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	Builder			100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:49:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:54:	MakeRequests		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:114:	MakeBids		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:149:	getMediaTypeForImp	100.0%
total:								(statements)		100.0%

@bsardo
Copy link
Collaborator

bsardo commented Nov 20, 2025

@Heihuang your tests are failing due to formatting issues. Please run go fmt.

@github-actions
Copy link

github-actions bot commented Nov 21, 2025

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, 9c8c4cb

iqiyi

Refer here for heat map coverage report

github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:21:	pickCurrency		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:33:	Builder			100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:49:	buildEndpointURL	100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:54:	MakeRequests		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:114:	MakeBids		100.0%
github.com/prebid/prebid-server/v3/adapters/iqiyi/iqiyi.go:149:	getMediaTypeForImp	100.0%
total:								(statements)		100.0%

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

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

adapter

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Heihuang @bsardo