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

Empty path masks field causes issues #111

Closed
BrenekH opened this issue Sep 10, 2021 · 11 comments
Closed

Empty path masks field causes issues #111

BrenekH opened this issue Sep 10, 2021 · 11 comments
Labels
bug Something isn't working

Comments

@BrenekH
Copy link
Owner

BrenekH commented Sep 10, 2021

When the path masks field is left empty, it matches against every file.

@BrenekH BrenekH added the bug Something isn't working label Sep 10, 2021
@BrenekH BrenekH added the waiting on merge Complete but not yet merged into master label Sep 11, 2021
@BrenekH BrenekH mentioned this issue Sep 11, 2021
@BrenekH
Copy link
Owner Author

BrenekH commented Sep 15, 2021

This has been released in 0.3.1.

@BrenekH BrenekH closed this as completed Sep 15, 2021
@BrenekH BrenekH removed the waiting on merge Complete but not yet merged into master label Sep 15, 2021
@tr1p0p
Copy link

tr1p0p commented Sep 19, 2021

Seem the last release don't fix the issue for me, still have empty queue when i have path mask field empty. On a fresh install on a fresh server (Debian 10 arm64)

Edit : Confirm the logs still show : skipped because of a mask ()on the last encodarr controller 0.3.1
here my library config
Screenshot 2021-09-19 at 17-47-44 Libraries - Encodarr
:

@BrenekH
Copy link
Owner Author

BrenekH commented Sep 19, 2021

Since this was a client-side fix (in the WebUI), could you see if ignoring the cache while reloading (Ctrl + F5 on most browsers) fixes the issue. You will have to edit the library and then save it again.

Also, in the About section on the Settings page, does the Controller version say 0.3.1? I did have an issue when updating myself that I had to change the docker tag to 0.3.1 to get the right version.

@tr1p0p
Copy link

tr1p0p commented Sep 19, 2021

i didn't use docker just download arm64 build on release page. I try but don't seem to make a change, will see the logs. And yeah controller settings show 0.3.1

@BrenekH
Copy link
Owner Author

BrenekH commented Sep 19, 2021

Did it fix itself or are the logs still saying skipped?

@tr1p0p
Copy link

tr1p0p commented Sep 19, 2021

the logs still show the same error..

@tr1p0p
Copy link

tr1p0p commented Sep 20, 2021

I think you should re-open this issues to saw what's wrong.

@BrenekH BrenekH reopened this Sep 20, 2021
@tr1p0p
Copy link

tr1p0p commented Sep 21, 2021

Still have this also when add a thing into path mask : #112

So yeah, 0.3.1 don't solve anything to me.

@BrenekH
Copy link
Owner Author

BrenekH commented Sep 23, 2021

Fixes for #111 and #112 have been merged into develop. Can you check to make sure the binaries fix your issue before I release it?

@tr1p0p
Copy link

tr1p0p commented Sep 23, 2021

Still having SQL issues..

@BrenekH
Copy link
Owner Author

BrenekH commented Sep 23, 2021

Really? Even with a new database? The old one isn't going to work.

If it's still an issue with a new db, can you post the full error logs in #112.

@BrenekH BrenekH mentioned this issue Sep 27, 2021
@BrenekH BrenekH closed this as completed Sep 27, 2021
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

2 participants