search

vision10 / mssql-ssrs

Promise based api for MSSQL Reporting Services with ntlm, basic, custom security
23 stars 6 forks source link

Update reportExecution.js #10

Closed preethgit closed 4 years ago

preethgit commented 4 years ago

Basically I may not want the server URL to be bound to the report path as it is exclusive of the server where the application is hosted.

preethgit commented 4 years ago

I tried this in a new project where the serverurl by default is considered to be the local host and additionally it is adding up the SSRS url and ends up with 404 error. Kindly advise is there a better way to avoid this or my change can be accepted?

vision10 commented 4 years ago

Right now there's a need to set the report server with setServerUrl before using the getReportByUrl
The current configuration should work with the localhost server name. I did not work with localhost as servername

I would rather modify the existing function to resolve server url to empty string and assume its on localhost instead of creating a new function

preethgit commented 4 years ago

You can cancel the request for now, I have managed to arrive at the server URL right after commenting here. thank you very much for your reply, should we need any major changes, we will let you know.

Our goal is to invoke the SSRS reports from the report server using the credentials instead of passing it in the URL to the server. So we had chose this package which we believe is suitable, but still we did not complete our PoC. Should we need help, we will come back to you, kindly help us.