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

Does not try eSCL if WS fails #289

Closed
mikhailnov opened this issue May 18, 2023 · 2 comments
Closed

Does not try eSCL if WS fails #289

mikhailnov opened this issue May 18, 2023 · 2 comments

Comments

@mikhailnov
Copy link
Contributor

Hello!
I have tested sane-airscan with Kyocera ECOSYS M2640idw, which works via eSCL only by default.
When I had the wsdd (https://github.com/christgau/wsdd) daemon running, I got a message:
WSDD: bind(0.0.0.0:3702): Address is already in use
It is because wsdd listens 0.0.0.0:3702.
No scanners were found.
Then I added ws-discovery = off into /etc/sane.d/airscan.conf and eSCL Kyocera was found.
Seems that sane-airscan does not try eSCL if WS discovery fails, can it continue and try eSCL?

mikhailnov added a commit to mikhailnov/sane-airscan that referenced this issue May 18, 2023
mikhailnov added a commit to mikhailnov/sane-airscan that referenced this issue May 18, 2023
@alexpevzner
Copy link
Owner

Hi, @mikhailnov,

A while ago the similar issue was already reported to the WSDD project: christgau/wsdd#95

I was participating in the discussion and in the investigation

We came to agreement that issue is at their side, and issue seems to be fixed there. Strange, that it came back again.

The patch that you've proposed will lead to instability in the sane-airscan work. Sometimes it will work with WSD properly, sometimes it will not work. And it will be hard to guess for the end user what is going on.

In a case of conflict or if WS-Discovery is just needed, sane-airscan allows to disable WS-Discovery via its configuration file (/etc/sane.d/airscan.conf):

[options]
ws-discovery = off

@mikhailnov
Copy link
Contributor Author

Thanks! I had wsdd 0.6.4, it does not have that fix. Great if hackery in sane-airscan is not required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants