Skip to content
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

OTT-48 VAST Bidder Phase 1 #186

Merged
merged 5 commits into from
Jul 27, 2021
Merged

OTT-48 VAST Bidder Phase 1 #186

merged 5 commits into from
Jul 27, 2021

Conversation

pm-viral-vala
Copy link
Collaborator

No description provided.

pm-viral-vala and others added 5 commits July 26, 2021 19:31
@pm-viral-vala
Squashed commit of the following:
bae9228 
commit 4be0c87
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Jul 26 18:38:47 2021 +0530

    Adding test cases for domain filtering and commenting filter vast tags for adpod

commit 07d1b4c
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Thu Jul 22 15:46:44 2021 +0530

        OTT-241 URL Encoding All Values of Macro

commit 5911dd0
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jul 20 14:21:52 2021 +0530

    merged master branch into OTT-48

commit e6fb5fb
Author: PubMatic-OpenWrap <UOEDev@pubmatic.com>
Date:   Fri Jul 16 00:34:00 2021 +0530

    OTT-233 : Fix error code for VAST bidders when bid price is zero

commit 6160f64
Author: mohammad-murad <mohammad.murad@pubmatic.com>
Date:   Thu Jul 15 11:10:50 2021 +0530

    OTT-232 : Fix incorrect tagID in final VastTag URLs

    Signed-off-by: mohammad-murad <mohammad.murad@pubmatic.com>

commit 708f2e6
Author: PubMatic-OpenWrap <UOEDev@pubmatic.com>
Date:   Mon Jul 5 15:26:27 2021 +0530

    OTT-217 - Remove logger for excluded VASTTags

commit d755118
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Wed Jun 30 14:27:09 2021 +0530

    OTT-48: Fixing Panic Issue

commit fa61146
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jun 29 15:21:25 2021 +0530

    Resolving Test Cases

commit db2b45a
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Jun 28 22:23:25 2021 +0530

    OTT-162 Adding BlockedVASTTags in Bid Response

commit f1f8712
Author: mohammad-murad <mohammad.murad@pubmatic.com>
Date:   Mon Jun 28 17:43:49 2021 +0530

    OTT-214 : Fix panic while getting debug info http calls

    Signed-off-by: mohammad-murad <mohammad.murad@pubmatic.com>

commit ff2b05c
Author: mohammad-murad <mohammad.murad@pubmatic.com>
Date:   Wed Jun 23 14:23:40 2021 +0530

    OTT:161 - Filter VAST tags based on duration

    Signed-off-by: mohammad-murad <mohammad.murad@pubmatic.com>

commit 58d0a4e
Merge: fb6d3c2 ca9b117
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Jun 18 20:03:35 2021 +0530

    Merge branch 'master' into OTT-48

commit fb6d3c2
Author: ShriprasadM <shriprasad.marathe@gmail.com>
Date:   Mon Jun 7 17:07:47 2021 +0530

    OTT-205: Added temporary change for enable QA to debug and verify the changes for OTT-159 (#166)

    Co-authored-by: Shriprasad <shriprasad.marathe@pubmatic.com>

commit 1def51c
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Thu May 20 10:14:22 2021 +0530

    OTT-48 Fixing formatting of OTT-48 based on master branch

commit cafa88f
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue May 18 22:47:03 2021 +0530

    OTT-48 Using PubMatic-OpenWrap etree library for vast xml processing

commit 101f131
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue May 18 20:52:05 2021 +0530

    merged master branch into OTT-48

commit 6789fa2
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon May 17 19:05:38 2021 +0530

    OTT-159,OTT-160 Adding Duration and VASTTag Details in ExtBidPrebidVideo Object

commit bd352c2
Author: Viral Vala <63396712+pm-viral-vala@users.noreply.github.com>
Date:   Mon May 3 14:15:55 2021 +0530

    OTT-57,OTT-153 Mocking VAST Tag Details and Update Request Handler fo… (#147)

    * OTT-57,OTT-153 Mocking VAST Tag Details and Update Request Handler for VAST Tag

    * OTT-57 Incorporating Review Comments

commit 8388840
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Apr 16 10:01:52 2021 +0530

    resolving master merge issue

commit 4a88c61
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Apr 13 12:19:28 2021 +0530

    Resolving Merge Issue

commit c01ab29
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Apr 12 14:29:22 2021 +0530

    merging master to OTT-48 branch

commit d61bacb
Author: mohammad-murad <mohammad.murad@pubmatic.com>
Date:   Tue Feb 23 16:54:33 2021 +0530

    OTT-100 : Get advertiser list from VAST

    Signed-off-by: mohammad-murad <mohammad.murad@pubmatic.com>

commit 3db8442
Author: Shriprasad <shriprasad.marathe@pubmatic.com>
Date:   Tue Feb 16 16:18:37 2021 +0530

    OTT-48: Enabled running all tests for TestApplyAdvertiserBlocking and corrected syntax error

commit 54b073e
Merge: b76d1a4 6fcee64
Author: Shriprasad <shriprasad.marathe@pubmatic.com>
Date:   Tue Feb 16 16:02:49 2021 +0530

    Merge branch 'master' into OTT-48

commit b76d1a4
Author: ShriprasadM <shriprasad.marathe@gmail.com>
Date:   Wed Feb 10 16:40:09 2021 +0530

    OTT-101: Added exchange.applyAdvertiserBlocking and unit tests around it (#108)

    * OTT-101: Added exchange.applyAdvertiserBlocking and unit tests around it

    * OTT-101: Unit tests added with tested workflow. Integrated the function

    * OTT-101: Removed check

    * OTT-101: Corrected test

    * OTT-101: Removed publicsuffix.tld+1. Instead we will return domain irrepstive of level.
    Renamed adjustDomain to normalizeDomain. Added and modified few test cases

    * OTT-101: Temporary method vastbidder.getAdvertisers  and its integration added for making this drop testable.
    OTT-100 must replace this function and its integration with valid implementation
    Also change the way of checking if bidder is instance of vasttag bidder. There was issue in determining it
    while doing the integration testing

    * OTT-101: Added new bucket in adaptermap - tagbidders. This map will hold the information
    around tagbidders. This bucket would be used by ApplyAdvertiserBlocking function for checking if bid belongs to ApplyAdvertiserBlocking.
    Also Replaced Index method with HasPrefix and also added case-insenstive comparison

    * OTT-101: Change the way of detecting the bidder instance. Now bidder will be resolved using resolveBidder method based on alias

    * OTT-101: Added publicsuffix check

    * OTT-101:Refectored the logic. Added more unit tests

    * OTT-101: Corrected the BidID and added one more bid request for testing

    Co-authored-by: Shriprasad <shriprasad.marathe@pubmatic.com>

commit 51dabf4
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Wed Feb 3 15:44:02 2021 +0530

    Merged 'master' branch into OTT-48

commit ea145cf
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Wed Jan 27 17:22:47 2021 +0530

    OTT-54 Updating longitude macro name from long to lon

commit 154a368
Author: ShriprasadM <shriprasad.marathe@gmail.com>
Date:   Wed Jan 27 15:21:36 2021 +0530

    OTT-88: Supporting Request Headers for Each VAST Bidder (#106)

    * OTT-88: Header changes

    * OTT-88: Added unit tests

    * OTT-88: Added support for VAST protocol based header set

    * OTT-88: Added unit test for checking if default + custom headers are set in requestData while making tag bidder request

    * OTT-88: Assigned value to IBidderMacro

    * OTT-88 fixing gofmt for vast bidder files

    * OTT-88: Added support for logging http request headers inside ext.debug.httpcalls.<bidder_>.requestheaders

    * Merge branch 'OTT-48' into OTT-88

    * OTT-88: Fixed test case issue

    * OTT-88: Reverted the comment

    * OTT-88: Code review changes are addressed

    * OTT-88: Refactored. Replace switch with if-else for better readbility. Added unit test when protocols contains vast 2 and vast 4 both

    Co-authored-by: Shriprasad <shriprasad.marathe@pubmatic.com>
    Co-authored-by: Viral Vala <viral.vala@pubmatic.com>

commit a8cb5f9
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 19 23:26:11 2021 +0530

    OTT-54 Updating scripts/coverage.sh to Skip prebid-server for coverage check

commit b390e5a
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 19 23:12:33 2021 +0530

    OTT-54 Fixing gofmt Formatting

commit 531671c
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 19 22:59:54 2021 +0530

    OTT-54: Adding Missing TestCase Files

commit 1fea95e
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 19 22:46:28 2021 +0530

    OTT-54 Adding TestCases

commit 74758d7
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 19 13:03:36 2021 +0530

    OTT-54 Fixing Bugs of Macro

commit c5e9530
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Jan 18 13:33:08 2021 +0530

    Merge branch 'master' into OTT-48

commit a17348d
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 12 12:17:54 2021 +0530

    Merging OTT-55 to OTT-48

commit 172b374
Merge: 776a1f3 e012115
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 12 12:12:57 2021 +0530

    Merge branch 'OTT-55' into OTT-48

commit e012115
Merge: 661db68 fde6dc4
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 12 12:10:42 2021 +0530

    Merge branch 'OTT-54' into OTT-55

commit fde6dc4
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Jan 12 12:00:04 2021 +0530

    OTT-54 Adding Default VAST Bidder Implementation

commit 776a1f3
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Jan 8 15:47:02 2021 +0530

    Merging OTT-54, OTT-55, OTT-58

    OTT-58: Extract Duration and Creative Id and update Prebid Server objects (#95)

commit 661db68
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Jan 8 10:30:14 2021 +0530

    OTT-54: Removing Bidder Config

commit c9e9c82
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Jan 8 10:27:22 2021 +0530

    removing bidder config

commit 12a8711
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Jan 4 16:41:24 2021 +0530

    OTT-55 Reverting Merge Conflicts

commit 90fe4c5
Merge: b17213c 9f69202
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Jan 4 14:28:40 2021 +0530

    Merge branch 'OTT-54' into OTT-55

commit 9f69202
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Dec 29 12:27:41 2020 +0530

    OTT-54 Adding Macro Support for Tag Bidder

commit b17213c
Author: ShriprasadM <shriprasad.marathe@gmail.com>
Date:   Mon Jan 4 12:39:46 2021 +0530

    OTT-58: Extract Duration and Creative Id and update Prebid Server objects (#95)

    * OTT-58: Added empty function getBidDuration. Added unit tests  around it

    * OTT-58: Few trials for converting VAST bid duration into seconds when input is  HH:MM:SS.mmm

    * OTT-58: Added unit tests and functionality for determining the video ad duration

    * OTT-58: Refactored and added some more unit tests

    * OTT-58: Added Benchmark testcase

    * OTT-58: Removed version argument from tests. Its not required for now. Fixed test issue

    * OTT-53: typo fix

    * OTT-58: Modifed regexp for millis to accept value max upto 999. Added unit tests around it

    * OTT-58: Added handling for detecting creative.id and updating it in typedBid.Bid.CrID

    * OTT-58: Addressed code review comments

    * OTT-58: Reverted changes for getCreativeID. It will now return empty string to creative id is not present.
    Instead caller will generate the random creative id with cr as prefix

    Co-authored-by: Shriprasad <shriprasad.marathe@pubmatic.com>

commit cfa29dc
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Sat Dec 19 09:36:38 2020 +0530

    added creative id parsing

commit bc87d85
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Sat Dec 19 09:15:39 2020 +0530

    updating macro processor

commit 40ebada
Author: Shriprasad <shriprasad.marathe@pubmatic.com>
Date:   Fri Dec 18 17:25:30 2020 +0530

    Revert "OTT-55: Added empty function getBidDuration. Added unit tests with expectations around it"

    This reverts commit 417f6fd.

commit 417f6fd
Author: Shriprasad <shriprasad.marathe@pubmatic.com>
Date:   Fri Dec 18 17:21:43 2020 +0530

    OTT-55: Added empty function getBidDuration. Added unit tests with expectations around it

commit ebcb221
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Dec 18 13:59:13 2020 +0530

    fixing bugs

commit 808264a
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Dec 18 11:33:09 2020 +0530

    updated macro format

commit dda1e87
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Fri Dec 18 11:11:45 2020 +0530

    testing phase

commit a439fd4
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Wed Dec 16 09:45:27 2020 +0530

    refactoring code

commit 5cbf91e
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Mon Dec 14 19:30:40 2020 +0530

    refactored code

commit 26c0619
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Sun Dec 13 00:22:19 2020 +0530

    refactoring code

commit 3dade2e
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Wed Dec 2 17:35:27 2020 +0530

    OTT-55 Adding Macro Definitions

commit 81841e5
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Tue Dec 1 17:45:31 2020 +0530

    OTT-55 Second Commit

commit 911db3e
Author: Viral Vala <viral.vala@pubmatic.com>
Date:   Wed Nov 25 12:28:34 2020 +0530

    OTT-55 First Commit
@pm-viral-vala
resolving master merge issues
21893b1
@pm-viral-vala
resolving master merge issue
8db27e2
@pm-viral-vala
Merge branch 'master' into OTT-48-master
cbe6fea
@pm-viral-vala
resolving gofmt for adapters/bidder.go
0fefc0a
@pm-viral-vala pm-viral-vala added the master_merge Indicates PR to be merged in master label Jul 27, 2021
@pm-viral-vala pm-viral-vala merged commit 3c564c7 into master Jul 27, 2021
sachin-pubmatic pushed a commit that referenced this pull request Aug 3, 2021
@pm-viral-vala @sachin-pubmatic
OTT-48 VAST Bidder Phase 1 (#186)
8eb3942
pm-nilesh-chate pushed a commit that referenced this pull request Jan 14, 2022
@pm-viral-vala @pm-nilesh-chate
OTT-48 VAST Bidder Phase 1 (#186)
26c4892
pm-nilesh-chate pushed a commit that referenced this pull request Jan 17, 2022
@pm-viral-vala @pm-nilesh-chate
OTT-48 VAST Bidder Phase 1 (#186)
d5a5870
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master_merge Indicates PR to be merged in master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pm-viral-vala