GithubHelp home page GithubHelp logo

Comments (16)

Airblader avatar Airblader commented on May 19, 2024 9

This assumes that those keys are bound to those functions, which they might not be. Hence this introduces a security risk. Ill close for that reason.

from i3lock.

aiguofer avatar aiguofer commented on May 19, 2024 7

Encountered this same issue. Found that the proposed commit presented here has been merged into https://github.com/PandorasFox/i3lock-color so you can just pass the --pass-media-keys option.

Hooray for forks and open source :) thanks everyone!

from i3lock.

jfrankenau avatar jfrankenau commented on May 19, 2024 3

If anyone is interested in a less hardcoded version than @bebehei's, have a look at 4d85b88. I have added an option to pass some XF86Audio* keys through to the root window by using xcb_send_event.

from i3lock.

stapelberg avatar stapelberg commented on May 19, 2024 2

We’d like to keep the option surface as small as possible, and i3lock as secure as possible.

Any feature (such as this one) which allows you to intentionally or accidentally weaken your computer’s security has to be carefully considered, and this one just doesn’t tip the scales favorably enough for adding it, sorry.

from i3lock.

bebehei avatar bebehei commented on May 19, 2024 1

That's correct. Adding this without any configuration options will imply a security risk.

It would be great if you could give i3lock via options or configuration file some keystrokes, which are ok to pass to the wm while locked.

from i3lock.

bebehei avatar bebehei commented on May 19, 2024 1

@alecdwm No, this seems not to be fixed. But while viewing the changes, I got enough knowledge about i3lock.c to know where to patch it in a hardcoded manner.

At least it "works for me". See e4f2119

Thank you for your notification. I now had been able to fix it. I'm so happy.

from i3lock.

Airblader avatar Airblader commented on May 19, 2024

I don't think we want this kind of configurability in i3lock which we generally keep to be simple and safe. @stapelberg Agreed?

from i3lock.

stapelberg avatar stapelberg commented on May 19, 2024

Agreed.

@bebehei If you want features like this, a more full featured desktop environment/screen locker might provide them.

from i3lock.

avindra avatar avindra commented on May 19, 2024

Has anyone found a workaround for this? Or can suggest an alternative locker?

from i3lock.

alecdwm avatar alecdwm commented on May 19, 2024

FYI: this issue appears to have been fixed in i3lock-color

from i3lock.

bebehei avatar bebehei commented on May 19, 2024

@tsuflux Well, have you opened a pull request?

from i3lock.

luislobo avatar luislobo commented on May 19, 2024

@stapelberg @Airblader what about the commit added by @jfrankenau 4d85b88 ? Seems like an easy and secure way of handling the multimedia keys.

from i3lock.

stapelberg avatar stapelberg commented on May 19, 2024

It doesn’t address the fundamental concern originally stated in #52 (comment).

Feel free to use a fork with this feature, but for now, I don’t think we want this in i3lock.

from i3lock.

luislobo avatar luislobo commented on May 19, 2024

@stapelberg well, I partially agree, but:

  • i3 is not run by normal regular users, normally people that know what they are doing
  • the change in the commit needs a particular specific parameter to be set so that i3lock passes through those keys, it's not the "standard" mode.
  • If an i3 kind of user doesn't know what their multimedia keys are doing (specially since you have to specifically configure the multimedia keys in i3 for them to work)... well, you are already in trouble.

from i3lock.

kpcyrd avatar kpcyrd commented on May 19, 2024

I think the original concern by @Airblader was about passing those keys by default. Since 4d85b88 is opt-in, the only risk is somebody explicitly enabling this feature without testing what those keys do on their setup. I think being able to quickly mute audio is a very reasonable request.

from i3lock.

luislobo avatar luislobo commented on May 19, 2024

That's fine. I'll just fork it.

from i3lock.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.