Skip to content

Commit 919fc54

Browse files
authored
Merge pull request #564 from splitio/fix-http-split-fetcher
Fix HttpSplitFetcher
2 parents 1acf543 + 4ae42b9 commit 919fc54

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

client/src/main/java/io/split/client/HttpSplitChangeFetcher.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public final class HttpSplitChangeFetcher implements SplitChangeFetcher {
4040
private static final String SETS = "sets";
4141
private static final String SPEC = "s";
4242
private String specVersion = SPEC_1_3;
43-
private int PROXY_CHECK_INTERVAL_MILLISECONDS_SS = 24 * 60 * 60 * 1000;
43+
private int PROXY_CHECK_INTERVAL_MILLISECONDS_SS = 24 * 60 * 60 * 1000;
4444
private Long _lastProxyCheckTimestamp = 0L;
4545
private final SplitHttpClient _client;
4646
private final URI _target;
@@ -70,11 +70,13 @@ long makeRandomTill() {
7070
public SplitChange fetch(long since, long sinceRBS, FetchOptions options) {
7171
long start = System.currentTimeMillis();
7272
try {
73+
URI uri = buildURL(options, since, sinceRBS);
7374
if (specVersion.equals(SPEC_1_1) && (System.currentTimeMillis() - _lastProxyCheckTimestamp >= PROXY_CHECK_INTERVAL_MILLISECONDS_SS)) {
7475
_log.info("Switching to new Feature flag spec ({}) and fetching.", SPEC_1_3);
7576
specVersion = SPEC_1_3;
77+
uri = buildURL(options, -1,-1);
7678
}
77-
URI uri = buildURL(options, since, sinceRBS);
79+
7880
SplitHttpResponse response = _client.get(uri, options, null);
7981
if (response.statusCode() < HttpStatus.SC_OK || response.statusCode() >= HttpStatus.SC_MULTIPLE_CHOICES) {
8082
if (response.statusCode() == HttpStatus.SC_REQUEST_URI_TOO_LONG) {

0 commit comments

Comments
 (0)