Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent Responses #121

Closed
siunus opened this issue Aug 15, 2024 · 5 comments
Closed

Inconsistent Responses #121

siunus opened this issue Aug 15, 2024 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@siunus
Copy link

siunus commented Aug 15, 2024

Version of RouterOS
RouterOS 6.48.3 (stable)

To Reproduce
Running a query like the one below sometimes yields unexpected responses:

$query = (new Query('/ppp/secret/print'));
$res = $client->query($query)->read(false);

Expected behavior
The correct response should look like this:
image

However, after refreshing a couple of times, I got this instead:
image

The parsed result is cut off in this section:
image

Could someone explain where the problem might be?

@siunus siunus added the bug Something isn't working label Aug 15, 2024
@EvilFreelancer
Copy link
Owner

Hi! Looks like a bug on the RouterOS side, which version of ROS do you use? Did you tried other versions?

@somzoli
Copy link
Contributor

somzoli commented Sep 19, 2024

I have the same issue here. /export results sometimes missing the end of the full export. If i run the export on router terminal, the output is full, but when i run the qury in router-os api, the results sometimes not contain the last X line.
We have huge configurations, so the export contains about ~60000 line.

@somzoli
Copy link
Contributor

somzoli commented Sep 19, 2024

In ssh (from my pc), the export works as expected. Seems its a [routeros-api-php] issue.

@somzoli
Copy link
Contributor

somzoli commented Sep 20, 2024

For export (ssh), ive made a pull request for change executeAsync to execute. In ssh export this caused early quit from loop.

@EvilFreelancer
Copy link
Owner

Hello! Thanks, PR merged, release tagged as 1.5.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants