systopia / CiviProxy

A security proxy for CiviCRM
GNU Affero General Public License v3.0
7 stars 18 forks source link

Extension does not support -dev release #46

Closed codebymikey closed 3 years ago

codebymikey commented 3 years ago

The regex at https://github.com/systopia/CiviProxy/blob/bbf5c3c9e9c0b3d8e453cc2139d0ab17d84dba59/de.systopia.civiproxy/CRM/Admin/Form/Setting/ProxySettings.php#L108-L113 does not match dev releases such as "CiviProxy Version 0.6-dev" which is the current latest version.

bjendres commented 3 years ago

That's a good point. @codebymikey Care to provide a PR?

codebymikey commented 3 years ago

Made a PR, the regex could be simplified to (([0-9]+(\.[0-9]+){1,2})(?:-[0-9A-Za-z-]+)?), but that's up to you.