Hi Gerado,

Go to: https://issues.apache.org/jira/projects/PLC4X/summary and create an
account with your mail address. Login and then you can create an issue in
our jira.

Thank you for supporting our project. :)

Kind regards
Lukas

Am Mi., 22. Juli 2020 um 19:17 Uhr schrieb Gerardo Melia <
[email protected]>:

> Hi  Julian, thanks for the reply. I'm not sure about how to open an issue.
> Sorry :(
>
> On Mon, 20 Jul 2020 at 17:00, Julian Feinauer <
> [email protected]>
> wrote:
>
> > Hi Gerardo,
> >
> > Thanks for your mail!
> > This indeed is a bug as far as I see it.
> >
> > Would you mind opening an issue for it?
> > I will try to fix it tomorrow.
> >
> > Thanks!
> > Julian
> >
> > Holen Sie sich Outlook für Android<https://aka.ms/ghei36>
> >
> > ________________________________
> > From: Gerardo Melia <[email protected]>
> > Sent: Monday, July 20, 2020 5:02:58 PM
> > To: [email protected] <[email protected]>
> > Subject: s7URIPattern doesn't match
> >
> > Hi people, first of all thanks for the amazing project that you are
> doing.
> > I have an issue with the 0.7.0 version when I'm using S7 driver. The
> regex
> > of the pattern doesn't match so I got this error: Caused by:
> > java.lang.IllegalArgumentException: s7://192.168.1.200/0/1<
> > http://192.168.1.200/0/1> doesn't match
> >
> >
> >
> >
> https://github.com/apache/plc4x/blob/1e6df7563ad909d365d1402729699b61c70019bd/plc4j/tools/connection-pool/src/main/java/org/apache/plc4x/java/utils/connectionpool/PoolKeyFactory.java#L46
> >
> > [image.png]
> >
> > This regex is correct but In version 0.7.0 I found this one in
> > PoolKeyFactory class:
> >
> > private final Pattern s7URIPattern =
> >
> Pattern.compile("^(?<poolablePart>s7://((?<ip>[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3})|(?<hostname>[a-zA-Z0-9\\.\\-]+))(:(?<port>[0-9]{1,5}))?)(?<params>\\?.*)?");
> >
> > that doesn't match : s7://192.168.1.200<http://192.168.1.200>/0/1 (With
> > no rack nor slot)
> >
> > Do you know if there is an issue here?
> >
> > Thanks in advance.
> >
> > Gerardo Melia
> >
> >
>

Reply via email to