Error fetching job: Invalid password - password with invalid URL characters

[ppires]

• Which Faktory package and version? Faktory 1.6.1 Docker image (contribsys/faktory:1.6.1)

• Which Faktory worker package and version? FWR 1.1.1

• Please include any relevant worker configuration

• Please include any relevant error messages or stacktraces

  /usr/local/bundle/gems/faktory_worker_ruby-1.1.1/lib/faktory/client.rb:352:in `result': Invalid password\r (Faktory::CommandError)
  from /usr/local/bundle/gems/faktory_worker_ruby-1.1.1/lib/faktory/client.rb:362:in `ok'
  from /usr/local/bundle/gems/faktory_worker_ruby-1.1.1/lib/faktory/client.rb:294:in `open'
  from /usr/local/bundle/gems/faktory_worker_ruby-1.1.1/lib/faktory/client.rb:58:in `initialize'
  from -e:1:in `new'
  from -e:1:in `<main>'

Are you using an old version? No

---

I'm using Faktory docker image version 1.6.1 (contribsys/faktory:1.6.1) and FWR 1.1.1

When I set a password for Faktory (using the env var FAKTORY_PASSWORD on the server) and the password has an invalid URL character (eg: @), when I connect to the server with FWR, I get the invalid password error. If I use a password without any URL invalid characters, everything works fine.

In the FAKTORY_URL I used the url-encoded password.

This docker compose file reproduces the error. If the password is changed to password (in both FAKTORY_PASSWORD and FAKTORY_URL), FWR can fetch jobs.

version: '3'
services:
  faktory:
    image: contribsys/faktory
    environment:
      - FAKTORY_PASSWORD=p@ssword
    command: /faktory

  example:
    image: ruby:2.7.6
    depends_on: [faktory]
    environment:
      - FAKTORY_URL=tcp://:p%40ssword@faktory:7419
    command: bash -c "gem install faktory_worker_ruby && ruby -r faktory_worker_ruby -e Faktory::Client.new"

Maybe a url-decode is missing before computing the password hash??

[mperham]

Send me a PR?

[mperham]

I think technically this issue belongs in fwr's repo.

[ppires]

I think technically this issue belongs in fwr's repo.

You're right, the problem is here, the password must be url-decoded before computing the hash.

Send me a PR?

It looks like a simple fix, I'll work on the PR. 😃

[ppires]

Migrated to https://github.com/contribsys/faktory_worker_ruby/issues/68