Actually this is not a bug.

We intentionally changed the sytax.
Please see the documentation page:
https://plc4x.apache.org/users/protocols/s7.html

We generally changed the connection string sytax for all drivers to a form that 
is more homogenous.

The reason for the change was that the pervious version you only contoured the 
local rack and slot and not the remote one.

Chris
________________________________
Von: Gerardo Melia <[email protected]>
Gesendet: Sonntag, 2. August 2020 18:16
An: [email protected] <[email protected]>
Betreff: Re: s7URIPattern doesn't match

Sorry about the delay. Here's the jira:

https://issues.apache.org/jira/browse/PLC4X-222

Regards

On Wed, 22 Jul 2020 at 14:31, Lukas Ott <[email protected]> wrote:

> 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