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.

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

Support useragent override in the constructor #26

Merged
merged 1 commit into from
Dec 8, 2020

Conversation

bmilekic
Copy link
Member

@bmilekic bmilekic commented Dec 7, 2020

When initializing the OptableSDK constructor, it's now possible to override the otherwise internally assigned user-agent string, with one specified by the caller. The caller-specified useragent is of course optional and, if one is not assigned (or nil is passed as the useragent at initialization time) then the default is to continue to create an invisible WebView and extract the user-agent string from it. If a caller specifies a non-nil useragent string, then we use it instead.

Closes #23

@bmilekic bmilekic force-pushed the 23-useragent-override branch from 1c5de45 to f7138f1 Compare December 7, 2020 23:56
When initializing the `OptableSDK` constructor, it's now possible to
override the otherwise internally assigned user-agent string, with one
specified by the caller. The caller-specified useragent is of course
optional and, if one is not assigned (or nil is passed as the useragent
at initialization time) then the default is to continue to create an
invisible WebView and extract the user-agent string from it. If a caller
specifies a non-nil useragent string, then we use it instead.

Closes #23

Documented in README
@bmilekic bmilekic force-pushed the 23-useragent-override branch from f7138f1 to f37f50c Compare December 8, 2020 03:09
@bmilekic bmilekic merged commit 442a5c2 into master Dec 8, 2020
@bmilekic bmilekic deleted the 23-useragent-override branch December 8, 2020 17:36
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.

Add ability to override useragent string at SDK initialization time
1 participant