-
Notifications
You must be signed in to change notification settings - Fork 13
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
lpvs fails without error #3
Comments
Good morning Andreas – sorry to hear you're having trouble. It sounds like you are able to execute `lpass export from Terminal? If so, the problem definitely lies with my side. Could you run |
Thanks Aaron, yes
|
Thank you. That all looks good. Let's go a level deeper:
Let me know what gets output and we'll go from there. |
Great instructions, thanks. The resulting output is the following:
Prettified:
|
Well that is very odd, indeed – that's exactly the XML I would expect. In fact, if I hardcode it into my Let's try one other thing: inside the
...to:
Any difference? |
Here's some debug output that goes with it:
|
Would you be willing to share your cash to data with me? You can get the path from that Alfred debug info. It only contains URLs and host names, so there shouldn't be anything damaging to you there. Feel free to email it to me at bachya1208@gmail.com. |
I'm a bit of an amateur when it comes to Terminal, but I can't seem to get past the python stage. I just get: -bash: python/Users/mattkneale/Library/Application Support/Alfred 2/Alfred.alfredpreferences/workflows/user.workflow.9E41EC43-4013-41AA-9C2C-A21A4B5B88D7/lpvm.py: No such file or directory EDIT: Scrap that, no space after python before dragging. Whoops! Here's what I get now: 16:04:24 workflow.py:2113 WARNING Can't save last version: workflow has no version |
Thanks, Matt! Looks like I could use some better debugging info. :) Hang tight while I look into some things; I may have both you and Andreas try a pre-release build with more debugging. |
No problem :) Thanks for your work. FWIW I love the look of the workflow, and it's still pretty cool for generating passwords quickly on the fly so still useful even if we can't get it resolved! |
Appreciate that! We'll get this squashed, don't worry – I want you to experience the full benefits of this workflow. :) Could you confirm whether your |
Let's hope so! /usr/bin/env python, and can confirm it makes no difference if I remove the /usr/bin/env. |
@avallen – I loaded the data you sent my into my cache and was able to query it just fine: So, this tells me that the issue might be environmental. Still experimenting, but wanted to let you know. |
@avallen and @MattKneale – I've updated the 3.2 pre-release with more debugging; would you mind:
Thanks! |
I tried but did not receive any debug output from
Could it be that the pre-release version that is offered under the above link still points to the previous version - it says: bachya released this 2 days ago. |
I'll try this later tonight/early tomorrow :) Just out at the moment. On 6 January 2015 at 20:12, Andreas Vallen notifications@github.com wrote:
|
@avallen It's the same release; I just added a different That's interesting that you don't see any debug output from either; you should definitely be seeing XML and other results. So, we've figured out that:
Is there anything special about your environment? What version of OS X and Python are you using? |
No idea RE: python (can't even remember installing it; how do I find the I'm on the beta channel for Yosemite... probably should have mentioned that On Tue, 6 Jan 2015 22:06 Aaron Bach notifications@github.com wrote:
|
In Terminal, run:
Interesting... @avallen, what about you? Are you on a Yosemite beta, as well? I'm running full-fledged Yosemite... I wonder if that's a problem. |
Version is 2.7.6. Yosemite is 10.10.2 On Tue, 6 Jan 2015 23:51 Aaron Bach notifications@github.com wrote:
|
No, I'm not on the Yosemite beta channel, instead running the regular current Yosemite version (10.10.1). Python version in Terminal (with PATH having brew's /usr/local/bin prepended) is:
Not sure about the python version in a launchd environment however, if alfred uses that, it might pick up the system python under /usr/bin instead. |
Hi Aaron, I've installed 3.2 with no success regretfully. Could you clarify how to access the debug outputs of the scripts? Still a novice to this! |
Well guys, this is stumping me. Interestingly enough, I'm starting to notice it happen on my side, too: I'll type I'm currently working on a complete rewrite that makes it much cleaner to see where things are going on. Once it's ready, I'll report back here and would love to have you test. Assuming it continues, I'm going to exercise the nuclear option and ask the Alfred forum participants for help. :) Stay tuned! |
No problem, sorry it didn't work as planned! Look forward to seeing the Best, Matt
|
@bachya that's a problem, if you do lpass --version it should answer 0.3.0... but try also brew info lastpass-cli it should give you 0.4.1. Knowing the fact that both can't be installed... I just think is a bug of the brew package. |
For what it's worth, I have exactly the same behavior as @markerx's summary. |
@bdsivak: That seems to indicate something going wrong with Alfred, then. :-\ Have you tried the new 2.6 version? |
@bachya I am using 2.6 since its release yesterday. No help. |
@bachya I've found the solution 👍 (at least for the update/no result error) So, the only solution, is to enter the vault from the website and check if any of your websites repromt for master password. Another solution is to run the download-data process not in background, but directly in the terminal, in order to let the user insert the password Hope that is a good solution which helps lot of us... |
@giulios123: WOW – great detective work! It makes sense that I would not be able to see this, as I don't have any sites for which I require a master password re-entry. @avallen @MattKneale @shkm @AnxiousGummi @nicatronTg @markerx: What about you all? Do you have items in your vault that require/force a master password re-entry? |
@bachya - I have over 1200 passwords... but I don't recall using the "Require Password Reprompt". @giulios123 - Do you know of a way to filter/identify any passwords using "Require Password Reprompt"? |
|
HI Aaron, yes I certainly have some!
|
@giulios123 What screen is that? I don't see a padlock anywhere. |
@giulios123 - nevermind. I found it. The Google LP Vault does not have the "Require Password Reprompt" column available. But FireFox does. @bachya - I found 2 entries that required "Require Password Reprompt". I removed that option from both. I did a:
All of those work. LPVS does not :-( |
@markerx: What happens if you open a Terminal and run |
A thousand sites scroll up the terminal window. |
@bachya - For what it is worth, lpbrowser works. |
@markerx: Well that's scary: |
@bachya I definitely have a few like that, including form fills. On 29 January 2015 at 00:52, markerx notifications@github.com wrote:
|
Ok, I've tried some more stuff...
Let me know if this works... ( on mine it worked perfectly). Or, if you want to keep the reprompt active you have to run *python path_to_workflow/lpdd_exec.py download-data each time you want to update the vault. @bachya one question, can you switch the process of lpass export from subprocess to a terminal window(it should workout fully with whom has the pass reprompt)? |
Arg... so close... Here's where I am at:
Fail :-( lpbrowser still works |
@markerx try in terminal python path_to_workflow/lpdd_exec.py download-data N: path_to_workflow is where lpdd_exec is |
/usr/bin/python /Users/Brandon/Library/Application\ Support/Alfred\ 2/Alfred.alfredpreferences/workflows/user.workflow.3791B45A-EA77-477F-9386-5C7A5A56F5E7/lpdd_exec.py download-data
/usr/bin/python /Users/Brandon/Library/Application\ Support/Alfred\ 2/Alfred.alfredpreferences/workflows/user.workflow.3791B45A-EA77-477F-9386-5C7A5A56F5E7/lpvs_query.py search-vault-for-query 'github'
|
so it's working? because it seems so eventually... log out again from terminal and kill all the lpass processes |
It works from a terminal stand point, but there seems to be a disconnect in the Alfred LPVS function. |
@giulios123 From your feedback, I've seen something interesting. I used to use At any rate, it appears that that's where the hang is. From what you've reported, I think I need a smarter way to check whether a user is already logged in or not. Researching... |
I would suggest lpass sync, which syncronize the local cache with the server one... but usually no change happens (due to the fact that we can't save from the workflow), so it just give no output if logged in and error if not logged :) |
@giulios123 The only problem with that (or any I think I've landed on checking for the existence of Here's the next problem: if I detect that the user isn't logged in, what would be the preferred action? If I notify the user (via Alfred) that they aren't logged in, that's great, but then what? |
I would rather automatically open the terminal for the insertion of the master pass ( and second factor if required). What do you think? |
It would be, I was going to play around with unix "expect" this weekend to Just got busy this week after spending too much time researching earlier. Wayne A Arthurton http://warthurton.com On Thu, Jan 29, 2015 at 5:25 PM, Giulio notifications@github.com wrote:
|
Hi Aaron,
many thanks for your great work, I look forward to making extensive use of it in the future, saving me some minutes per day! :-)
Now everything seems to work fine save the most important thing:
lpvs <term>
does just not respond, instead it just displays "Searching vault...". Nothing is output in alfred's debug mode (set to output all information).The data that returned from
lpass export
does contain non-ASCII characters, so this might be related to issue #2, testing with the 3.2 pre-release that you offer in that issue does not fix the issue however (after re-executing lpdd).I'd be glad to provide additional debugging help if you'd like to investigate this further.
Thanks,
Andreas
The text was updated successfully, but these errors were encountered: