Skip to content
2 changes: 1 addition & 1 deletion app/Console/Commands/RebuildQueryserviceData.php
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public function handle(MediaWikiHostResolver $mwHostResolver) {
$skippedWikis = 0;
$processedWikis = 0;
foreach ($wikis as $wiki) {
$this->apiUrl = $mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php'; // used in PageFetcher::fetchPagesInNamespace
$this->apiUrl = $mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php'; // used in PageFetcher::fetchPagesInNamespace

try {
$entities = $this->getEntitiesForWiki($wiki);
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/CirrusSearch/CirrusSearchJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ public function handle(HttpRequest $request, MediaWikiHostResolver $mwHostResolv

$request->setOptions(
[
CURLOPT_URL => $mwHostResolver->getBackendHostForDomain($this->wiki->domain) . '/w/api.php?action=' . $this->apiModule() . $this->getQueryParams(),
CURLOPT_URL => $mwHostResolver->getBackendUrlForDomain($this->wiki->domain) . '/w/api.php?action=' . $this->apiModule() . $this->getQueryParams(),
CURLOPT_RETURNTRANSFER => true,
CURLOPT_ENCODING => '',
CURLOPT_TIMEOUT => $this->getRequestTimeout(),
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/MediawikiInit.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ public function handle(HttpRequest $request, MediaWikiHostResolver $mwHostResolv
];

$request->setOptions([
CURLOPT_URL => $mwHostResolver->getBackendHostForDomain($this->wikiDomain) . '/w/api.php?action=wbstackInit&format=json',
CURLOPT_URL => $mwHostResolver->getBackendUrlForDomain($this->wikiDomain) . '/w/api.php?action=wbstackInit&format=json',
CURLOPT_RETURNTRANSFER => true,
CURLOPT_ENCODING => '',
CURLOPT_TIMEOUT => 60,
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/MediawikiSandboxLoadData.php
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ public function handle(MediaWikiHostResolver $mwHostResolver) {

$curl = curl_init();
curl_setopt_array($curl, [
CURLOPT_URL => $mwHostResolver->getBackendHostForDomain($this->wikiDomain) . '/w/rest.php/wikibase-exampledata/v0/load',
CURLOPT_URL => $mwHostResolver->getBackendUrlForDomain($this->wikiDomain) . '/w/rest.php/wikibase-exampledata/v0/load',
CURLOPT_RETURNTRANSFER => true,
CURLOPT_ENCODING => '',
CURLOPT_TIMEOUT => 10 * 60, // TODO Long 10 mins (probably shouldn't keep the request open...)
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/PlatformStatsSummaryJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ public function prepareStats(array $allStats, $wikis): array {
$currentTime = CarbonImmutable::now();

foreach ($wikis as $wiki) {
$this->apiUrl = $this->mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php'; // used in PageFetcher::fetchPagesInNamespace
$this->apiUrl = $this->mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php'; // used in PageFetcher::fetchPagesInNamespace

if (!is_null($wiki->deleted_at)) {
$deletedWikis[] = $wiki;
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/PollForMediaWikiJobsJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ private function hasPendingJobs(string $wikiDomain): bool {
$response = Http::withHeaders([
'host' => $wikiDomain,
])->get(
$this->mwHostResolver->getBackendHostForDomain($wikiDomain) . '/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json'
$this->mwHostResolver->getBackendUrlForDomain($wikiDomain) . '/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json'
);

if ($response->failed()) {
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/SiteStatsUpdateJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ public function handle(HttpRequest $request, MediaWikiHostResolver $mwHostResolv
Log::info(__METHOD__ . ": Updating stats for $wiki->domain");

$request->setOptions([
CURLOPT_URL => $mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php?action=wbstackSiteStatsUpdate&format=json',
CURLOPT_URL => $mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php?action=wbstackSiteStatsUpdate&format=json',
CURLOPT_RETURNTRANSFER => true,
CURLOPT_ENCODING => '',
CURLOPT_TIMEOUT => 60 * 5,
Expand Down
10 changes: 5 additions & 5 deletions app/Jobs/UpdateWikiSiteStatsJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ private function updateSiteStats(Wiki $wiki): void {
$response = Http::withHeaders([
'host' => $wiki->getAttribute('domain'),
])->get(
$this->mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json'
$this->mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json'
);

if ($response->failed()) {
Expand All @@ -81,7 +81,7 @@ private function updateSiteStats(Wiki $wiki): void {

private function getFirstEditedDate(Wiki $wiki): ?CarbonInterface {
$allRevisions = Http::withHeaders(['host' => $wiki->getAttribute('domain')])->get(
$this->mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php',
$this->mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php',
[
'action' => 'query',
'format' => 'json',
Expand All @@ -99,7 +99,7 @@ private function getFirstEditedDate(Wiki $wiki): ?CarbonInterface {
}

$revisionInfo = Http::withHeaders(['host' => $wiki->getAttribute('domain')])->get(
$this->mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php',
$this->mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php',
[
'action' => 'query',
'format' => 'json',
Expand All @@ -119,7 +119,7 @@ private function getFirstEditedDate(Wiki $wiki): ?CarbonInterface {

private function getLastEditedDate(Wiki $wiki): ?CarbonInterface {
$allRevisions = Http::withHeaders(['host' => $wiki->getAttribute('domain')])->get(
$this->mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php',
$this->mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php',
[
'action' => 'query',
'format' => 'json',
Expand All @@ -137,7 +137,7 @@ private function getLastEditedDate(Wiki $wiki): ?CarbonInterface {
}

$revisionInfo = Http::withHeaders(['host' => $wiki->getAttribute('domain')])->get(
$this->mwHostResolver->getBackendHostForDomain($wiki->domain) . '/w/api.php',
$this->mwHostResolver->getBackendUrlForDomain($wiki->domain) . '/w/api.php',
[
'action' => 'query',
'format' => 'json',
Expand Down
2 changes: 1 addition & 1 deletion app/Jobs/WikiEntityImportJob.php
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ private static function domainToOrigin(string $domain): string {

private static function acquireCredentials(string $wikiDomain, MediaWikiHostResolver $mwHostResolver): OAuthCredentials {
$response = Http::withHeaders(['host' => $wikiDomain])->asForm()->post(
$mwHostResolver->getBackendHostForDomain($wikiDomain) . '/w/api.php?action=wbstackPlatformOauthGet&format=json',
$mwHostResolver->getBackendUrlForDomain($wikiDomain) . '/w/api.php?action=wbstackPlatformOauthGet&format=json',
[
'consumerName' => 'WikiEntityImportJob',
'ownerOnly' => '1',
Expand Down
4 changes: 4 additions & 0 deletions app/Services/MediaWikiHostResolver.php
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,10 @@ public function getBackendHostForDomain(string $domain): string {
return sprintf('mediawiki-%s-app-backend.default.svc.cluster.local', $this->getMwVersionForDomain($domain));
}

public function getBackendUrlForDomain(string $domain): string {
return 'http://' . $this->getBackendHostForDomain($domain);
}

private function getMwVersionForDomain(string $domain): string {
$wiki = Wiki::where('domain', $domain)->first();

Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"absszero/laravel-stackdriver-error-reporting": "^1.9",
"firebase/php-jwt": "^7.0",
"google/recaptcha": "^1.2",
"guzzlehttp/guzzle": "^7.8",
"guzzlehttp/guzzle": "^7.13",
"guzzlehttp/psr7": "^2.9",
"hackzilla/password-generator": "^1.6",
"intervention/image": "^2.5",
Expand Down
64 changes: 33 additions & 31 deletions composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions tests/Commands/RebuildQueryserviceDataTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,10 @@ protected function setUp(): void {
WikiSetting::query()->delete();
QueryserviceNamespace::query()->delete();

$this->mwBackendHost = 'mediawiki.localhost';
$this->mwBackendHost = 'http://mediawiki.localhost';

$mockMwHostResolver = $this->createMock(MediaWikiHostResolver::class);
$mockMwHostResolver->method('getBackendHostForDomain')->willReturn(
$mockMwHostResolver->method('getBackendUrlForDomain')->willReturn(
$this->mwBackendHost
);

Expand Down
4 changes: 2 additions & 2 deletions tests/Jobs/CirrusSearch/ForceSearchIndexTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,10 @@ protected function setUp(): void {
'wiki_id' => $this->wiki->id,
]);

$this->mwBackendHost = 'mediawiki.localhost';
$this->mwBackendHost = 'http://mediawiki.localhost';

$this->mockMwHostResolver = $this->createMock(MediaWikiHostResolver::class);
$this->mockMwHostResolver->method('getBackendHostForDomain')->willReturn(
$this->mockMwHostResolver->method('getBackendUrlForDomain')->willReturn(
$this->mwBackendHost
);
}
Expand Down
4 changes: 2 additions & 2 deletions tests/Jobs/CirrusSearch/QueueSearchIndexBatchesTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -47,10 +47,10 @@ protected function setUp(): void {
'wiki_id' => $this->wiki->id,
]);

$this->mwBackendHost = 'mediawiki.localhost';
$this->mwBackendHost = 'http://mediawiki.localhost';

$this->mockMwHostResolver = $this->createMock(MediaWikiHostResolver::class);
$this->mockMwHostResolver->method('getBackendHostForDomain')->willReturn(
$this->mockMwHostResolver->method('getBackendUrlForDomain')->willReturn(
$this->mwBackendHost
);
}
Expand Down
4 changes: 2 additions & 2 deletions tests/Jobs/MediawikiInitTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,10 @@ protected function setUp(): void {
$this->wikiDomain = 'some.domain.com';
$this->username = 'username';
$this->email = 'some@email.com';
$this->mwBackendHost = 'mediawiki.localhost';
$this->mwBackendHost = 'http://mediawiki.localhost';

$this->mockMwHostResolver = $this->createMock(MediaWikiHostResolver::class);
$this->mockMwHostResolver->method('getBackendHostForDomain')->willReturn(
$this->mockMwHostResolver->method('getBackendUrlForDomain')->willReturn(
$this->mwBackendHost
);
}
Expand Down
4 changes: 2 additions & 2 deletions tests/Jobs/PlatformStatsSummaryJobTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,10 @@ protected function setUp(): void {
$this->wikis = [];
$this->users = [];

$this->mwBackendHost = 'mediawiki.localhost';
$this->mwBackendHost = 'http://mediawiki.localhost';

$this->mockMwHostResolver = $this->createMock(MediaWikiHostResolver::class);
$this->mockMwHostResolver->method('getBackendHostForDomain')->willReturn(
$this->mockMwHostResolver->method('getBackendUrlForDomain')->willReturn(
$this->mwBackendHost
);
}
Expand Down
Loading
Loading