Error 400 Invalid URI calling SOA Function

itSDS · 2023-02-07 12:07:15

Hi Arnaud,

with yesterday tests i experience an Error 400 calling SOA Functions.

This Error comes not with Version 4496 but exists with Version 4665.
Sry the Repository Version between this 2 i don't have.

If you need more input I'll try to provide it.

itSDS · 2023-02-07 12:11:34

One more Info: We use the Syncrossplatform Client from m1 to access the SOA Functions

ab · 2023-02-07 13:39:52

Try to add '/' in the beginning of all requested URI:

procedure TSQLRestClientURI.URI(var Call: TSQLRestURIParams);
...
  if (Call.Url<>'') and (Call.Url[1]<>'/') then
    Call.Url := '/'+Call.Url;
  InternalURI(Call);
...

If it is not enough, please post here the actual values of Call.Url from the client side.
And try to debug on the server side why it is not routed as expected, e.g. in TRestRouter.Lookup().

itSDS · 2023-02-07 15:43:16

Hi Arnaud, i see TRestRouter.Lookup is new

There is the Problem, it does not find my Function (GetStammdaten) But Stammdaten is in the tree under Get (GetStammdaten, GetArtikel, ...) i have 4 Methods starting with get.

The Value of Call URL is : 'LSYSConnector/DFLSYSConnector.GetStammdaten?session_signature=000e153ce78f9d46a320304d'

Last edited by itSDS (2023-02-07 15:49:16)

itSDS · 2023-02-07 15:53:40

Here is the matching RadixTree:

https://postimg.cc/zHr9fz6V

May be the Backslash at stammdaten/ ?!

Last edited by itSDS (2023-02-07 15:54:27)

itSDS · 2023-02-07 16:13:19

Trying to add / at the Beginning does not work. On Server lsysconnector/timestamp -> //lsysconnector/timestamp if i add your code sample.

ab · 2023-02-07 18:06:51

The tree seems correct: you need to look at FullUrl field to see the actual URI.
Because at each node level, you only have the new text in Chars[].
So there is a 'get' node, with 'stammdaten' sub-node for 'getstammdaten' URI.

itSDS · 2023-02-07 18:15:19

You can see the fullurl in the bottom line of my Screenshot.
There is a slash at the end. That's the reason why Lookup fails. In Requested URL is no slash at the end but ?session...
In the former Version 4495 the String was accecpted.

I think it's a missing feature in new Lookup Method - what do you think ?

Last edited by itSDS (2023-02-07 18:18:05)

ab · 2023-02-07 18:38:05

Where does this trailing '/' come from?

On my side, TRestServerRoutingRest.UriComputeRoutes register the routes with no trailing /

Please post on a gist your full tree content:
You can just uncomment line 6829 of mormot.rest.server - writeln(r.Tree[mPOST].ToText);

For instance, here is a tree during SOA regression tests: https://gist.github.com/synopse/8b994c0 … 1dce6296eb

itSDS · 2023-02-08 08:14:31

Here is the Tree:

lsysconnector/
              dflsysconnector
                             /
                              get
                                 a
                                  usgabemengen/
                                               <clientid>
                                  rtikel/
                                         <clientid>
                                 stammdaten/
                                            <clientid>
                                 inventur/
                                          <clientid>
                                 belege/
                                        <clientid>
                              sende
                                   inventur/
                                            <clientid>
                                   artikel/
                                           <clientid>
                                   belege/
                                          <clientid>
                              druckeb
                                     arcode/
                                            <clientid>
                                     eleg/
                                          <clientid>
                              _
                               free_/
                                     <clientid>
                               instance_
                               contract_
                             .
                              get
                                 a
                                  usgabemengen/
                                               <clientid>
                                  rtikel/
                                         <clientid>
                                 stammdaten/
                                            <clientid>
                                 inventur/
                                          <clientid>
                                 belege/
                                        <clientid>
                              sende
                                   inventur/
                                            <clientid>
                                   artikel/
                                           <clientid>
                                   belege/
                                          <clientid>
                              _
                               free_/
                                     <clientid>
                               contract_
                               instance_
                              druckeb
                                     arcode/
                                            <clientid>
                                     eleg/
                                          <clientid>
              cacheflush
                        /
                         <fulluri>
              wrapper
                     /
                      <fulluri>
              batch

itSDS · 2023-02-08 08:24:13

Here the deklaration part

  IDFLSYSConnector = interface(IInvokable)
    ['{DCBB296A-C209-4346-BD37-4496E32A50BE}']
    function GetStammdaten(const AAuthToken : RawUtf8; const AGeaendertNach : TDateTime = 0) : RStammdatenResult;
...

ab · 2023-02-08 09:36:41

In sicClientDriven mode?
My guess is that it should rather be in sicShared mode on server side.

Your previous code worked by chance with the cross-platform clients.
You should specify sicClientDriven also on client side, to have the trailing /<clientid> specified in the client URI.

itSDS · 2023-02-08 12:35:36

The Problem is only with the Syncrossplatform Client. The normal m2 Client works.
I use sicClientDriven on both Sides.

I just changed sicClientDriven on Server Side to sicShared, and the Error is gone.
Are there any things i have to change on the Server now ?
Is this the right way - Change Mode on Server ?

itSDS · 2023-02-08 14:05:38

Sry i have to add this, i read the Doku for sicShared / sicClientDriven again.
I need sicClientDriven on the Server Settings, my Server Implementation affords this. There are Client specific var i share between all "Client" requests. The Server Implmentation is needed for every client.

Do i have to provide the Client ID now in SynCrossplatformclient ? and how ?

itSDS · 2023-02-08 14:19:39

Now i checked the URL called from one of the m2 client and you add e.g. stammdaten/1?session_ .. as ClientID to the request where is this clientid generated ? i think it should be on the Server or ?

itSDS · 2023-02-08 18:28:15

Hi Arnaud,

what should i do now.
I have to use SynCrossplatformclient on my App - Client Side (it is the only client i have atm). But i have to use it with sicClientDriven on Client and Server side. (or am i wrong ?)
You have enhanced m2 with a clientID. But i don't know how to enhance Syncrossplatformclient to use/get it.

atm im lost here ?!

ab · 2023-02-08 19:09:30

And it works by chance/error before in your case: without any /<clientid> the server does not know which server instance to use, so it should reject it.
As the new code does.

Try to look on the client side why the /<clientid> trailer is not transmitted to the server.
Perhaps the mORMot 1 cross-platform client does not make what is required.

In sicClientDriven mode, the clientid is a number returned by the server when /_instance_ is called.
It will identify the server class instance to use for executing the service method for this unique client.
Then, at client shutdown, in sicCLientDriven mode, the client should call /_free_/<clientid> to release the server class instance.

itSDS · 2023-02-08 19:19:32

Ok ty i have enhanced the SynCrossplatform Client to ask for instance and use Result as ClientID now.

mORMot Open Source

#1 2023-02-07 12:07:15

Error 400 Invalid URI calling SOA Function

#2 2023-02-07 12:11:34

Re: Error 400 Invalid URI calling SOA Function

#3 2023-02-07 13:39:52

Re: Error 400 Invalid URI calling SOA Function

#4 2023-02-07 15:43:16

Re: Error 400 Invalid URI calling SOA Function

#5 2023-02-07 15:53:40

Re: Error 400 Invalid URI calling SOA Function

#6 2023-02-07 16:13:19

Re: Error 400 Invalid URI calling SOA Function

#7 2023-02-07 18:06:51

Re: Error 400 Invalid URI calling SOA Function

#8 2023-02-07 18:15:19

Re: Error 400 Invalid URI calling SOA Function

#9 2023-02-07 18:38:05

Re: Error 400 Invalid URI calling SOA Function

#10 2023-02-08 08:14:31

Re: Error 400 Invalid URI calling SOA Function

#11 2023-02-08 08:24:13

Re: Error 400 Invalid URI calling SOA Function

#12 2023-02-08 09:36:41

Re: Error 400 Invalid URI calling SOA Function

#13 2023-02-08 12:35:36

Re: Error 400 Invalid URI calling SOA Function

#14 2023-02-08 14:05:38

Re: Error 400 Invalid URI calling SOA Function

#15 2023-02-08 14:19:39

Re: Error 400 Invalid URI calling SOA Function

#16 2023-02-08 18:28:15

Re: Error 400 Invalid URI calling SOA Function

#17 2023-02-08 19:09:30

Re: Error 400 Invalid URI calling SOA Function

#18 2023-02-08 19:19:32

Re: Error 400 Invalid URI calling SOA Function

Board footer