Modify ↓

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#2718 closed defect (duplicate)

[patch] unclosed regex group

Reported by: schuetzm@… Owned by: xeen
Priority: critical Milestone:
Component: Core imagery Version:
Keywords: Cc:

Description

The regex
(?:mapwarper\.net|warper\.geothings\.net/(?:.*?)/([0-9]+)(?:\?|/|\.|$)
is not properly closed, which make the wmsplugin fail to load:
Caused by: java.util.regex.PatternSyntaxException: Unclosed group near index 70

This was introduced in changeset 15827.

Attachments (1)

fixwms.patch​ (809 bytes ) - added by xeen 16 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 by xeen, 16 years ago

Strange, why did it compile for me?

Fix is pretty easy, new regex should be

(?:mapwarper\.net|warper\.geothings\.net)/(?:.*?)/([0-9]+)(?:\?|/|\.|$)

I can submit a patch later, but I don't have a checked out source at hand right now.

comment:2 by anonymous, 16 years ago

Owner: changed from team to xeen

by xeen, 16 years ago

Attachment: fixwms.patch​ added

comment:3 by anonymous, 16 years ago

Summary: unclosed regex group → [patch] unclosed regex group

comment:4 by xeen, 16 years ago

WARNING: Patch included in #2719 ! (i.e. don't check this one in)

comment:5 by anonymous, 16 years ago

Resolution: → duplicate
Status: new → closed

comment:6 by stoecker, 16 years ago

Ticket #2720 has been marked as a duplicate of this ticket.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain xeen.
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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.