Rework Optable API configuration and request forming #35

ValentinPostindustria · 2025-11-17T13:57:23Z

Closes #33.

Breaking changes:

Remove app path parameter.
Add tenant and originSlug query parameters.
Make host dynamic and apply path v2.

Other features:

Support apiKey.
Move all configuration parameters in Config class and reuse it everywhere.
Separate GoogleAdIdManager and LocalStorage logic from the network client.
Rename Client to NetworkClient.
Move hash logic to separate TypeHasher class.
Separate network interceptors.
Demo apps are built using local SDK.
Unit tests with mock web server.
Build WebView for user agent only once and cache the result.
Apply automatic IDEA aligning.

Remove `app` path parameter. Add `tenant` and `originSlug` parameters. Make host dynamic and apply path `v2`. Move all configuration parameters in Config class and reuse it everywhere. Separate `GoogleAdIdManager` and `LocalStorage` logic from the network client. Rename `Client` to `NetworkClient`. Move hash logic to separate `TypeHasher` class. Separate network interceptors. Refactor SDK code.

Support secret api keys. Fetch user agent only once. Write unit tests with the mock web server.

android_sdk/src/main/java/co/optable/android_sdk/OptableSDK.kt

android_sdk/src/main/java/co/optable/android_sdk/core/UserAgentHolder.kt

android_sdk/src/main/java/co/optable/android_sdk/OptableSDK.kt

Remove custom network Adapter. Use native Retrofit adapter. Add Kotlin suspension.

Migrate from TypeHasher to IdentifiersEncoder. Add all identifier types. Cover with unit tests.

Use callbacks API. Rewrite demo apps to make the examples more readable. Use shorter logs. Create Optable SDK instance once during application creation and reuse everywhere. Move fragments to one folder.

Add missed "reg" query param. Make Consents mutable. Provide `receiveGaidAutomatically` property. Refactor main methods javadoc.

DemoApp/DemoAppJava/app/src/main/java/co/optable/demoappjava/TheApplication.java

DemoApp/DemoAppJava/app/src/main/java/co/optable/demoappjava/ui/GamBannerFragment.java

DemoApp/DemoAppJava/app/src/main/java/co/optable/demoappjava/ui/PrebidBannerFragment.java

DemoApp/DemoAppKotlin/app/src/main/java/co/optable/androidsdkdemo/ui/GamBannerFragment.kt

android_sdk/src/main/java/co/optable/android_sdk/OptableSDK.kt

android_sdk/src/main/java/co/optable/android_sdk/core/IdentifiersEncoder.kt

android_sdk/src/main/java/co/optable/android_sdk/OptableIdentifiers.kt

android_sdk/src/test/java/co/optable/android_sdk/OptableIdentifiersTest.kt

justadreamer

we are almost there, a couple minor changes mentioned in the comments:

we need to preserve original targetingData instead of completely transforming it to gamTargetingKeywords and ortbJson - it may still be used for other purposes
we need to make sure we use keyspace, not provider as a key in the gamTargetingKeywords - maybe I looked in the wrong place - it seemed to me provider was used
please make sure that in all cases when skipAdvertisingIdentifer==false we obtain gaid automatically

android_sdk/src/main/java/co/optable/android_sdk/OptableSDK.kt

justadreamer · 2026-01-13T13:20:03Z

android_sdk/src/main/java/co/optable/android_sdk/OptableSDK.kt

+        } else null
+
+        val gaid = if (gaid) {
+            adIdManager.getId()


do we pull gaid automatically in case of any identify call when skipAdvertisingIdDetection == false?

DemoApp/DemoAppJava/app/src/main/java/co/optable/demoappjava/ui/PrebidBannerFragment.java

DemoApp/DemoAppJava/app/src/main/java/co/optable/demoappjava/ui/GamBannerFragment.java

justadreamer · 2026-01-13T15:12:45Z

android_sdk/src/main/java/co/optable/android_sdk/OptableIdentifiers.kt

+    val receiveGaidAutomatically: Boolean = false,
+) {
+
+    private val cachedEnrichedIds: List<String> by lazy { IdentifiersEncoder.encode(this) }


is "enriched ids" their original term?

No. It's a new field, it wasn't before. I took it from iOS.
They called it: idList and type-prefixed identifiers. So I renamed to EIDs.

android_sdk/src/main/java/co/optable/android_sdk/OptableTargeting.kt

ValentinPostindustria added 5 commits November 17, 2025 14:53

fix: unit tests

8d6e1de

feat: new unique suffix

7efd435

feat: use local SDK sources by default

dc6b91f

feat: api key and testing

71fe63a

Support secret api keys. Fetch user agent only once. Write unit tests with the mock web server.

ValentinPostindustria marked this pull request as ready for review November 21, 2025 09:07

ValentinPostindustria requested a review from zapo November 21, 2025 09:08

justadreamer requested changes Dec 15, 2025

View reviewed changes

ValentinPostindustria added 2 commits December 15, 2025 13:47

feat: introduce OptableConfig

ad8359b

fix: unit tests

5ae14c5

ValentinPostindustria requested a review from justadreamer December 15, 2025 13:01

ValentinPostindustria and others added 15 commits December 18, 2025 12:35

Introduce Optable demo cases with Prebid (#32)

ec71ba5

feat: refactor network layer

c6381b3

feat: introduce callbacks for identify and targeting methods

dd4ad43

feat: rewrite demo apps to use callbacks

63ba731

refactor: examples

674a426

feat: rewrite network logic

bbb6638

Remove custom network Adapter. Use native Retrofit adapter. Add Kotlin suspension.

feat: introduce OptableIdentifiers

0ee30b3

Migrate from TypeHasher to IdentifiersEncoder. Add all identifier types. Cover with unit tests.

feat: Rx LiveData to traditional callbacks

ff2bbcf

feat: targeting keywords for GAM

ade2a26

feat: refactor demo apps

fd0300d

Use callbacks API. Rewrite demo apps to make the examples more readable. Use shorter logs. Create Optable SDK instance once during application creation and reuse everywhere. Move fragments to one folder.

feat: support consents

07c7ff4

refactor: some methods

8e464dc

docs: rewrite README file

3aff74c

test: consent query params

2b8c7eb

feat: corrections

bf7a416

Add missed "reg" query param. Make Consents mutable. Provide `receiveGaidAutomatically` property. Refactor main methods javadoc.

justadreamer requested changes Jan 11, 2026

View reviewed changes

ValentinPostindustria added 3 commits January 12, 2026 20:02

feat: rename audiences to gamTargetingKeywords

6f8e459

fix: use keyspace instead of provider for the key of GAM targeting

2f98ab2

refactor: rename generateEnrichedIds

f8c4255

refactor: eidFromUrl method

8b2e4da

justadreamer requested changes Jan 13, 2026

View reviewed changes

ValentinPostindustria added 3 commits January 13, 2026 20:22

feat: automatic GAID by default

4a1c99f

feat: targetingData field in OptableTargeting

066b592

refactor: rename enrichedIds

d369331

ValentinPostindustria requested a review from justadreamer January 13, 2026 19:31

Rework Optable API configuration and request forming #35

Are you sure you want to change the base?

Rework Optable API configuration and request forming #35

Uh oh!

Conversation

ValentinPostindustria commented Nov 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

justadreamer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

justadreamer Jan 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

justadreamer Jan 13, 2026

Choose a reason for hiding this comment

Uh oh!

ValentinPostindustria Jan 13, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ValentinPostindustria commented Nov 17, 2025 •

edited

Loading

justadreamer Jan 13, 2026 •

edited

Loading