Modify

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#17240 closed enhancement (fixed)

Presets: verify that wiki links are no redirect

Reported by: simon04 Owned by: Don-vip
Priority: minor Milestone: 19.10
Component: Unit tests Version:
Keywords: preset redirect http osm-wiki Cc: Don-vip, stoecker

Description (last modified by Klumbumbus)

From ticket:17173#comment:15 Don-vip:

Replying to ticket:17173#comment:14 stoecker:

It should not be default to produce useless redirects for each wiki call.

We should add an integration test to check this.

-> after r15423 Jenkins revealed:

https://josm.openstreetmap.de/jenkins/job/JOSM-Integration/3725/testReport/

github issues see comment:9

Attachments (0)

Change History (20)

comment:1 by Don-vip, 6 years ago

Component: CoreUnit tests

comment:2 by Don-vip, 5 years ago

Owner: changed from team to Don-vip
Status: newassigned

comment:3 by Don-vip, 5 years ago

Milestone: 19.10

comment:4 by Don-vip, 5 years ago

Resolution: fixed
Status: assignedclosed

In 15423/josm:

fix #17240 - Presets: verify that wiki links are no redirect

comment:5 by Klumbumbus, 5 years ago

Description: modified (diff)

(typo)

comment:6 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:7 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:8 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:10 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:11 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:12 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:13 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:14 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:15 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:16 by stoecker, 5 years ago

Russian is also fixed. One remaining needs an ignore because of permanently changing URL. The other one seems also strange. See GitHub.

comment:17 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:18 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:20 by Klumbumbus, 5 years ago

The test currently returns:

[Found HTTP redirection for http://kolbasa.ru/where/ -> 301 -> https://kolbasa.ru/where/, Found HTTP redirection for https://samson-pharma.ru/pharmacies -> 301 -> http://samson-pharma.ru/pharmacies/]

For me https://samson-pharma.ru/pharmacies redirects to https://samson-pharma.ru/contacts/ instead.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Don-vip.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.