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
feat: Direct routing to correct pod #223
feat: Direct routing to correct pod #223
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
||
- (NSString *)defaultHostWithSubdomain:(NSString *)subdomain apiKey:(NSString *)apiKey enableDirectRouting:(BOOL)enableDirectRouting { | ||
if (enableDirectRouting) { | ||
NSArray *splitKey = [apiKey componentsSeparatedByString:@"-"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What would happen if the subdomian doesnt have the - for splitting?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if the API key doesn't have a dash (aka it's an old key without a prefix) this will just return an array with a single value, which I check for on the next line.
b0701a1
into
feat/SQDSDKS-5764-api-url-routing-feature-branch
Summary
Testing Plan
Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)