GP: Duplicate meta descriptions

src/routes/globalping.js

@@ -55,22 +55,53 @@ koaElasticUtils.addRoutes(router, [
  * Network tools pages.
  */
 koaElasticUtils.addRoutes(router, [
-	[ '/network-tools', '/network-tools/:params' ],
+	[ '/network-tools/:params?' ],
 ], async (ctx) => {
-	let data = {
-		params: ctx.params.params || '',
-	};
+	let data;
+	let { params = '' } = ctx.params;
+	let splitPoint = '-from-';
+	let splitPointIdx = params.indexOf(splitPoint);
+	let [ testType, target ] = splitPointIdx === -1
+		? params.split(splitPoint)
+		: [ params.slice(0, splitPointIdx), params.slice(splitPointIdx + splitPoint.length) ];
 
 	try {
+		// check if test type is correct
+		switch (testType.toLowerCase()) {
+			case 'ping':
+			case 'dns':
+			case 'mtr':
+			case 'http':
+			case 'traceroute':
+				data = {
+					params: ctx.params.params || '',
+				};
+
+				break;
+
+			default:
+				throw new Error(`Measurement type ${testType} is incorrect!`);
+		}
+
+		// if there is -from- part in the path but no target provided by user
+		if (splitPointIdx !== -1 && !target) {
+			throw new Error('Target was not provided!');
+		}
+
 		ctx.body = await ctx.render('pages/globalping/network-tools.html', data);
 		ctx.maxAge = 5 * 60;
 	} catch (e) {
-		if (app.env === 'development') {
-			console.error(e);
-		}
+		// TODO: 715 app call leas to Reference error: undefined
+		// if (app.env === 'development') {
+		// console.error(e);
+		// }
 
 		ctx.status = 301;
-		return ctx.redirect('/');
+
+		// redirect conserving the target or just to default ping test page
+		let newPath = target ? `ping-from-${target}` : 'ping';
+
+		return ctx.redirect(`/network-tools/${newPath}`);
 	}
 });
 

src/views/pages/globalping/docs/api.globalping.io.html

@@ -48,6 +48,7 @@
 		data () {
 			return {
 				title: 'Globalping API',
+				description: 'Use the Globalping API to access thousands of global probes and build your own tools and integrations.',
 			};
 		},
 		computed: {

src/views/pages/globalping/docs/auth.globalping.io.html

@@ -48,6 +48,7 @@
 		data () {
 			return {
 				title: 'Globalping Authentication API',
+				description: 'Authenticate your Globalping integrations using our OAuth API for the best user flow.',
 			};
 		},
 		computed: {

src/views/pages/globalping/network-tools.html

@@ -62,11 +62,11 @@ <h1>{{displayingTestType}} from {{#if locationData.probesCnt}}multiple locations
 
 						<span>test</span>
 					</div>
-				{{elseif placeholerLocation}}
+				{{elseif rawLocationToDisplay}}
 					<h1>{{displayingTestType}} from multiple locations in </h1>
 					<span class="gp-nettools_main-info_content_temp-location">
-						{{#if placeholerLocation.toLowerCase() !== 'world'}}
-							{{placeholerLocation}},
+						{{#if rawLocationToDisplay.toLowerCase() !== 'world'}}
+							{{rawLocationToDisplay}},
 						{{/if}}
 
 						<a target="_blank" rel="noopener noreferrer" href="{{global.location.origin}}/network-tools/{{selectedTestType}}-from-world" class="gp-nettools_green-text">
@@ -277,15 +277,30 @@ <h1>{{displayingTestType}} from multiple locations in </h1>
 				probesParsed: false,
 				testReqParams: null,
 				rawLocation: null,
+				rawLocationToDisplay: null,
 				allRequiredDataReady: false,
 			};
 		},
+		onconfig () {
+			if (!Ractive.isServer) {
+				let title, description;
+				let { rawTestType, rawLocation, rawLocationToDisplay } = this.getInitialRawDataFromUrl();
+
+				// create temp title, descr before any checks, required by GA
+				title = this.createMetaTitle(rawTestType, rawLocationToDisplay);
+				description = this.createMetaDescr(rawTestType, rawLocationToDisplay);
+
+				this.set('title', title);
+				this.set('description', description);
+				this.set('selectedTestType', rawTestType);
+				this.set('rawLocation', rawLocation);
+				this.set('rawLocationToDisplay', rawLocationToDisplay);
+			}
+		},
 		oninit () {
 			if (!Ractive.isServer) {
 				this.getGlobalpingProbesData();
 
-				this.getParamsFromUrl();
-
 				this.observe('selectedTestType', (selectedTestType) => {
 					if (selectedTestType) {
 						switch (selectedTestType.toLowerCase()) {
@@ -356,7 +371,7 @@ <h1>{{displayingTestType}} from multiple locations in </h1>
 					let rawLocation = this.get('rawLocation');
 
 					// handle locationData, highLevelLocationHref
-					if (rawLocation && rawLocation !== 'world') {
+					if (rawLocation && rawLocation.toLowerCase() !== 'world') {
 						let locationData = this.getLocDataByLocValue(rawLocation);
 
 						if (locationData) {
@@ -366,6 +381,8 @@ <h1>{{displayingTestType}} from multiple locations in </h1>
 						} else {
 							// when testType correct but location absent at the moment among the probes or incorrect
 							this.moveToNewPageURL(selectedTestType, null, true);
+
+							return;
 						}
 					} else {
 						this.set('locationData', { from: 'World', probesCnt: this.get('totalProbesCnt') });
@@ -374,7 +391,7 @@ <h1>{{displayingTestType}} from multiple locations in </h1>
 
 					// handle meta tag title and description once location is confirmed as presented
 					let locationData = this.getLocDataByLocValue(rawLocation);
-					let title = this.createMetaTitle(selectedTestType, locationData ? locationData.name : 'world');
+					let title = this.createMetaTitle(selectedTestType, locationData ? locationData.name : 'World');
 					let description = this.createMetaDescr(selectedTestType, locationData ? locationData.name : 'world');
 
 					this.set('title', title);
@@ -756,54 +773,6 @@ <h1>{{displayingTestType}} from multiple locations in </h1>
 				http.fetchGlobalpingProbes().then(this.handleProbesResponse(false));
 			}
 		},
-		getParamsFromUrl () {
-			let rawParamsValue = this.get('params');
-
-			// if user tries to get the network-tools page without providing at least testType
-			// navigate to default network-tools page
-			if (rawParamsValue === null || typeof rawParamsValue === 'undefined') {
-				this.moveToNewPageURL(INITIAL_OPTS_VALUES.type, null, true);
-
-				return;
-			}
-
-			let clearParamsValue = rawParamsValue.replace(/\/network-tools\/?/, '').toLowerCase();
-			let splitPoint = '-from-';
-			let splitPointIdx = clearParamsValue.indexOf(splitPoint);
-			let [ rawTestType, rawLocation ] = splitPointIdx === -1
-				? clearParamsValue.split(splitPoint)
-				: [ clearParamsValue.slice(0, splitPointIdx), clearParamsValue.slice(splitPointIdx + splitPoint.length) ];
-
-			// if testType provided by user is incorrect
-			// navigate to default network-tools testType page but conserve the location
-			if (!this.checkIfTestTypeCorrect(rawTestType)) {
-				let isFullParam = clearParamsValue.includes(splitPoint) && rawLocation && rawTestType;
-
-				this.moveToNewPageURL(INITIAL_OPTS_VALUES.type, isFullParam ? rawLocation : null, !isFullParam);
-
-				return;
-			}
-
-			// if testType provided by user is correct but there is no location provided
-			// navigate to provided by user testType network-tools page
-			if (clearParamsValue.includes(splitPoint) && !rawLocation) {
-				this.moveToNewPageURL(rawTestType, null, true);
-
-				return;
-			}
-
-			this.set('selectedTestType', rawTestType);
-			this.set('rawLocation', rawLocation || 'world');
-			let placeholderLocation = _.capitalizeStrEveryFirstLetter((rawLocation || 'world').split('-').join(' '), CAPITALIZE_EXCLUDE_LIST);
-			this.set('placeholerLocation', placeholderLocation);
-
-			// set temp meta tag title, descr
-			let title = this.createMetaTitle(rawTestType, placeholderLocation);
-			let description = this.createMetaDescr(rawTestType, placeholderLocation);
-
-			this.set('title', title);
-			this.set('description', description);
-		},
 		checkIfTestTypeCorrect (testType) {
 			if (!testType) { return false; }
 
@@ -886,5 +855,21 @@ <h1>{{displayingTestType}} from multiple locations in </h1>
 
 			return true;
 		},
+		getInitialRawDataFromUrl () {
+			let rawParamsValue = this.get('params');
+			let clearParamsValue = rawParamsValue.replace(/\/network-tools\/?/, '').toLowerCase();
+			let splitPoint = '-from-';
+			let splitPointIdx = clearParamsValue.indexOf(splitPoint);
+			let [ rawTestType, rawLocation ] = splitPointIdx === -1
+				? clearParamsValue.split(splitPoint)
+				: [ clearParamsValue.slice(0, splitPointIdx), clearParamsValue.slice(splitPointIdx + splitPoint.length) ];
+			let rawLocationToDisplay = _.capitalizeStrEveryFirstLetter((rawLocation || 'world').split('-').join(' '), CAPITALIZE_EXCLUDE_LIST);
+
+			return {
+				rawTestType,
+				rawLocation,
+				rawLocationToDisplay,
+			};
+		},
 	};
 </script>

src/views/pages/globalping/network.html

@@ -161,7 +161,7 @@
 		data () {
 			return {
 				title: 'Network - Globalping',
-				description: 'Network - Globalping',
+				description: 'Explore the global network map of Globalping probes.',
 				availableFilters: [
 					{
 						name: 'Europe',

src/views/pages/globalping/terms.html

@@ -100,6 +100,7 @@ <h1 class="page-content_main_title">Terms &amp; Policies</h1>
 		data () {
 			return {
 				title: 'Terms & Policies - Globalping',
+				description: 'Learn about the terms of usage of the global network testing platform Globalping.',
 				currentPolicy: 'terms-of-use',
 			};
 		},