- $hub_challenge = random_string(40);
- $params = 'hub.mode=' .
- ($subscribe == 1 ? 'subscribe' : 'unsubscribe') .
- '&hub.topic=' . urlencode($hub_topic) .
- '&hub.challenge=' . $hub_challenge .
- '&hub.lease_seconds=604800' .
- '&hub.verify_token=' . $hub_verify_token;
-
- // lease time is hard coded to one week (in seconds)
- // we don't actually enforce the lease time because GNU
- // Social/StatusNet doesn't honour it (yet)
-
- $fetchResult = Network::fetchUrlFull($hub_callback . "?" . $params);
+ $hub_challenge = Strings::getRandomHex(40);
+
+ $params = http_build_query([
+ 'hub.mode' => $subscribe == 1 ? 'subscribe' : 'unsubscribe',
+ 'hub.topic' => $hub_topic,
+ 'hub.challenge' => $hub_challenge,
+ 'hub.verify_token' => $hub_verify_token,
+
+ // lease time is hard coded to one week (in seconds)
+ // we don't actually enforce the lease time because GNU
+ // Social/StatusNet doesn't honour it (yet)
+ 'hub.lease_seconds' => 604800,
+ ]);
+
+ $hub_callback = rtrim($hub_callback, ' ?&#');
+ $separator = parse_url($hub_callback, PHP_URL_QUERY) === null ? '?' : '&';
+
+ $fetchResult = Network::fetchUrlFull($hub_callback . $separator . $params);