I never authorized an integration that scans my secrets

(github.com)

97 points | by trvrprkr an hour ago ago

48 comments

  • frenck 43 minutes ago

    Frenck here, I'm the one who replied on that issue.

    My replies there were more dismissive than they should have been. My apologies. I've posted a follow-up on the issue.

    As it turns out, the check can already be turned off after all:

      ha security options --pwned=false
    
    It has been part of the Supervisor for years and checks app options that are marked as passwords. A recent release fixed the warning that surfaces it, which is why people are noticing it now.

    Only the first 5 characters of a SHA-1 hash go to HIBP, and the matching happens locally: https://github.com/home-assistant/supervisor/blob/main/super...

    Our documentation should have explained all of this, including how to turn it off. We'll get that updated.

    • infinite_spin 8 minutes ago

      I do get the sense that you're not malicious, just that maybe you're not seeing this from the standpoint of someone that assigns a higher value to some elements of privacy. In this case, your user base expects a home automation system. They do not expect that system to scan their computer for passwords, regardless of how "safely" the home automation system were to behave.

      You might have a heart of gold, and just be having an off day due to reasons, so I hope people cut you some slack, but if I were you I'd update this feature to be opt-in and conspicuous, immediately

    • saghm 17 minutes ago

      Asking basically the same thing as the sibling replies, but hopefully in a less aggressive way: can you elaborate on what led you to reply like that in the first place? As someone with no horse in the race (I don't use Home Assistant), the first reply you gave genuinely does seem like it was intended in good faith, even if it was not satisfying to the user, and I can understand that angry users are not fun to interact with, but I'm struggling to wrap my head around why your second reply was "This issue is closed as won't fix" without any elaboration about why. The explanation that seems most likely to me is that you were annoyed at the user and decided not to spend any time looking into the issue, which would be unfortunate but maybe understandable if that's what you're apologizing for. Without any explanation about why you quickly dismissed and then suddenly turned around and changed your mind later though, I think you've risked the apology coming across more as damage control than genuine regret.

    • 5kyl4b 26 minutes ago

      why lying on your first reply? you mentioned 1Password did the same, in an attempt to make yourself "less guilty", but you failed to realize 1Password doesn't enable that option automatically without consent, and in some cases this option is periodically DISABLED automatically. address this, please, it's pretty embarrassing to lie like this just to cover y'all's ass.

      • hypfer 18 minutes ago

        From my understanding, they seem to be under a lot of stress/pressure lately.

        I'm not exactly sure why, but the stuff that happens tells that story (to me at least).

        Barking at frenck on a Friday at 11.30PM like this is probably not the right thing to do.

        • frenck 17 minutes ago

          Hehe not really. I'm also just enjoying my hobby on a Saturday evening honestly :)

          But you are right, it is time to head downstairs for a beer with Mrs. Frenck.

          ../Frenck

      • frenck 21 minutes ago

        Not a lie, but I was wrong on that one. I'm happy to admit that.

        I've used 1Password's Watchtower for so many years that I assumed the breached password check was just part of it. After your reply I looked it up and learned it's actually a setting in their privacy preferences, so my comparison was off.

        ../Frenck

    • omega3 16 minutes ago

      > My replies there were more dismissive than they should have been. My apologies.

      I’m curious what made you change your mind, perhaps seeing this getting traction on hn?

    • philipwhiuk 23 minutes ago

      > Only the first 5 characters of a SHA-1 hash go to HIBP

      Some would consider the first five characters being sent over the internet to be 'your secret leaving your system'.

      Some would consider phoning home to a remote server to be worrisome. It's home assistant not cloud assistant.

    • doobiedowner 35 minutes ago

      >As it turns out, the check can already be turned off

      Problem solved!

      >My replies there were more dismissive than they should have been

      What about this one?

  • janice1999 an hour ago

    Toxic developers turned me off Home Assistant. The same developer is the star of this issue: https://github.com/NixOS/nixpkgs/pull/126326

    Related discussion: https://news.ycombinator.com/item?id=27505277

    • fg137 10 minutes ago

      Was thinking of setting my own assistant using Home Assistant to replace the Google Home device I got rid of. Looks like I should slow down and take a closer look at this.

      As someone whose smart home need is minimal -- I just use it to turn on/off lights with voice -- I'd rather have nothing than support such behavior.

    • someguydave 7 minutes ago

      interesting that the crux of the Nix dispute was about how he wanted to self-modify the HA after install, and the crux of the current dispute is how HA should be able to scan your secrets and upload truncated hashes

      It's not a good look taken together

    • hypfer 9 minutes ago

      Dude that is

      a) five years ago

      b) nothing to be ashamed of, honestly

      c) dumb drama

      This is bad faith. You could reference it in good faith as context, but that is not what you are doing.

      • jeremyjh 2 minutes ago

        I found it helpful and informative.

    • netsharc 32 minutes ago

      Hah, his profile sure has a lot of bullshit about being human and positivity...

  • jackpepsi an hour ago

    You may find this unasked for scan humiliating, but we can mathematically prove we're not humiliating you. Any feelings of discomfort or humiliation are therefore invalid and should be set aside.

    Edit: I'm sure a greater good argument can be made. I just don't think they are making it in a way that justifies the invasiveness.

  • interf4ce an hour ago

    It sounds like the dev's reason for rejecting the issue is that the issue was opened on the documentation repo rather than the code repo.

    On the one hand, it's understandable. No amount of documentation changes will fix the issue.

    On the other hand, it's a pretty abrasive response. It wouldn't have taken any more effort to be a little more helpful and direct OP to the right repo: "Sorry, this is the wrong place for this to be addressed. Please bring it up here instead [link]."

    • mcmcmc 21 minutes ago

      If you actually read through, the option to disable the functionality was already present… just not documented. So it seems the documentation repo was the correct place to open it

  • altairprime an hour ago

    (For missing context from this bare link to an issue with no context provided by the submitter — Home Assistant apparently integrated the same Have I Been Pwned safe scanning process that many other password managers have.)

    I have a great deal of sympathy for them refusing to engage; decisive software creators are far less palatable to an entitled swarm of disagreement, and that same decisiveness is what makes the best software. Perhaps in U.S. culture, refusal with a smile and a reassurance that ā€œWe’re Listeningā€ is something folks feel entitled to, but in other areas of the world, demanding a stranger change their mind once it’s made up earns only mockery or pity. To those cultures, this GitHub is a comedy-farce, and I laughed out loud upon reading it — not because I necessarily agree with the project’s decision, but because I agree culturally with their tone and handling of what some view as blatant immaturity: a final flat refusal, closing the door, and getting back to work.

  • TavsiE9s an hour ago

    Tough one. I get the intention but this should've been opt-in.

  • TZubiri 35 minutes ago

    >>please let us know so we can uninstall and fork

    >Sorry, these are not statements we can work with. Doing such threats and trying to intimidate it not going to bring you anywhere. If you want to do a

    just wanted to add an outsider perspective here, not trying to add wood to the fire

    but i don’t think it’s inappropriate or ā€˜a threat’ to let the dev know the user can stop using the software.

    • dpark 24 minutes ago

      Is it inappropriate? Not really. Is it a threat? Yeah, it really is. ā€œI’m going to stop using your software if you don’t change thisā€ is a threat. ā€œI’m going to fork it and ship a competing productā€ is a stronger threat.

      It was a clear attempt to sway with intimidation. It was done politely but that doesn’t change what it was. (The dev had already closed as won’t fix, too, so they had preemptively answered.)

  • csoghoian an hour ago

    Giving the home assistant core developers the benefit of the doubt, I am sure there are good intentions here. But if they are interested in adding new security features, adding support for phishing-resistant multi factor authentication (passkeys) would be much more valuable. Users have been asking for it for some time, and even submitted patches, which were rejected.

  • thunderfork 20 minutes ago

    So, wait, these are secrets that are already given to h-a, hashed, and just the first five characters of the hash sent up..?

    Yeah, I agree with the maintainer, this strikes me as outrage theater over any actual problem, no?

  • Rygian an hour ago

    I wouldn't be surprised if that integration is outright illegal in EU due to GDPR, thus putting HIBP in trouble for ingesting PII without any legal legitimacy.

    And the first reply being "It is not an invasion of privacy" just shows incompetence on privacy topics. It is never a valid answer.

    Which is really concerning, for the lead developer of a home automation system that "fights for the fundamental principle of privacy" as per their mission statement.

    • Matticus_Rex 27 minutes ago

      It's certainly an overstep, but just the hash they're sending wouldn't be PII.

    • deadbunny 23 minutes ago

      What PII do you think is being sent to HIBP?

    • nicce 31 minutes ago

      I am curious thought that how it would be illegal according to GDPR. They are hashed after all, and the same hash is not used for authentication.

      • Rygian 4 minutes ago

        Undoing personal information for a purpose different than the purpose for which it was collected.

  • jocular 15 minutes ago

    Frenck is the worst part of Home Assistant. He shits on so many things with just a hand wave and no attempt at justification.

  • flanz_10 an hour ago

    > If this is all reason for you to uninstall and fork... So be it, it is open source after all.

    what a petty answer. "just fork it bro".

    • flapperdeflip 33 minutes ago

      Working on a fork already: Lets spend some tokens! :)

  • abletonlive 30 minutes ago

    Here we have the classic if you want something to happen in the corporate world, make it go viral on the internet. Except, humans will be humans and it played out in open source.

    In this case, frenck most certainly backtracked because they realized they were on the front page of HN and did not want the heat.

  • onionisafruit an hour ago

    I have another tab open where I am about to buy a home assistant green and start a journey of probably-pointless automations. This is giving me pause. Not about sending hashes to haveibeenpwned, but about the response this got on github.

  • Cameri an hour ago

    The dev comes as self-serving. Not what you’d expect to see. Calls it heated when he himself caused it with their tone deaf response… it’s next level.

    • deadbunny 35 minutes ago

      The same dev doesn't even like others packaging his open source libs.

      https://github.com/NixOS/nixpkgs/pull/126326

    • csnover an hour ago

      > The dev comes as self-serving

      What, do you not include links to every single one of your social media accounts in every single post you make on GitHub? I think this is a very cool and normal thing, and not a concerning red flag about narcissistic tendencies at all.

  • eur0pa an hour ago

    Awfully tone deaf responses

  • l-albertovich an hour ago

    And that kids is a clear example of how not to interact with people online.

    • zephyreon an hour ago

      Yeah not a good look. This is probably the worst way to handle community feedback.

      • l-albertovich an hour ago

        I mean, none of them behaved like an adult.

        It's pretty clear that the maintainer should've handled it better but I'd be lying if I said I never felt like telling a petulant user off. The problem is he didn't handle it professionally.

        • gvfsa 36 minutes ago

          Maybe he shouldn’t be accessing people’s passwords. If you start looking into someone’s wallet you can expect an abrasive response.

          • l-albertovich 26 minutes ago

            Well, I didn't check the git history so I can't confirm or deny that he implemented the feature, in any case, if you read the comment about how the passwords are anonymized it sounds pretty reasonable.

            Leaving that aside, even if you diagree with it you (the user) are not entitled to anything, especially not entitled to have an attitude.

            Just think about it differently, what do you think would happen if you do the same in the real world?

            I'm pretty sure in the best case scenario you may be asked to leave and in the worst case scenario you may be "escorted" out of the premises.

            A little bit of politeness goes a long way.

    • ragall 29 minutes ago

      The developer's reply was very reasonable and measured. The user, on the other hand, was very rude.