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

NO_REF: Experimental TFA support. #94

Closed
wants to merge 3 commits into from

Conversation

gdzevo
Copy link

@gdzevo gdzevo commented May 1, 2019

I'm not a pro in C++
Please be polite in comments :)

@blackjackshellac
Copy link

I managed to get it to compile and it runs but I don't know how to get a prompt for the TFA code.

@gdzevo
Copy link
Author

gdzevo commented May 1, 2019

@blackjackshellac have you enabled TFA for account?
First login after you enable TFA should be made in foreground (without -d flag).

@blackjackshellac
Copy link

I used used ploudcc -u [email protected] -p and it got stuck at CONNECTING

$ pcloudcc -u [email protected] -p
pCloud console client v.2.1.0
Please, enter password

Down: Everything Downloaded| Up: Everything Uploaded, status is LOGIN_REQUIRED
logging in
Down: Everything Downloaded| Up: Everything Uploaded, status is CONNECTING

After that nothing happens.

@blackjackshellac
Copy link

BTW, yes I did enable TFA. I just used to to reconnect the desktop version of pcloud 1.5.1

@gdzevo
Copy link
Author

gdzevo commented May 1, 2019

Hmm, that is how it should be:

Down: Everything Downloaded| Up: Everything Uploaded, status is LOGIN_REQUIRED
logging in
Down: Everything Downloaded| Up: Everything Uploaded, status is CONNECTING
Down: Everything Downloaded| Up: Everything Uploaded, status is TFA_ERROR
two factor authentication - Please, enter password

Down: Everything Downloaded| Up: Everything Uploaded, status is CONNECTING
Down: Everything Downloaded| Up: Everything Uploaded, status is READY

Could you please check that you use latest libpcloudcc_lib.so and share system information and boost library version that you have used.
I'll try to reproduce this in docker.

@blackjackshellac
Copy link

Everything looks correct to me,

$ ll /usr/local/lib/libpcloudcc_lib.so 
-rwxr-xr-x 1 root root 5514424 May  1 10:34 /usr/local/lib/libpcloudcc_lib.so
$ ll /usr/local/bin/pcloudcc 
-rwxr-xr-x 1 root root 232680 May  1 10:34 /usr/local/bin/pcloudcc
$ ldd /usr/local/bin/pcloudcc 
	linux-vdso.so.1 (0x00007ffc509ab000)
	libpcloudcc_lib.so => /usr/local/lib/libpcloudcc_lib.so (0x00007fe188a68000)
	libboost_system.so.1.66.0 => /lib64/libboost_system.so.1.66.0 (0x00007fe188a61000)
	libboost_program_options.so.1.66.0 => /lib64/libboost_program_options.so.1.66.0 (0x00007fe1889de000)
	libfuse.so.2 => /lib64/libfuse.so.2 (0x00007fe18899d000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007fe18897b000)
	libz.so.1 => /lib64/libz.so.1 (0x00007fe188961000)
	libdl.so.2 => /lib64/libdl.so.2 (0x00007fe188959000)
	libstdc++.so.6 => /lib64/libstdc++.so.6 (0x00007fe1887c1000)
	libm.so.6 => /lib64/libm.so.6 (0x00007fe18863d000)
	libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007fe188622000)
	libc.so.6 => /lib64/libc.so.6 (0x00007fe18845c000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fe188ded000)
	librt.so.1 => /lib64/librt.so.1 (0x00007fe188452000)

@gdzevo
Copy link
Author

gdzevo commented May 1, 2019

@blackjackshellac finally compiled with boost 1.66 and it still working.
Which OS you use?

@blackjackshellac
Copy link

I was on Fedora 29.

@blackjackshellac
Copy link

I updated to Fedora 30 in the interrim and this is what I see now,

pCloud console client v.2.1.0
Please, enter password
Down: Everything Downloaded| Up: Everything Uploaded, status is SCANNING
Down: Everything Downloaded| Up: Everything Uploaded, status is READY

But it never asks me for a token ... maybe it's because I was using the desktop client. I'll try on another machine.

@blackjackshellac
Copy link

This is what I had to change to get it to compile on my system, can you do a git apply to test on your branch,

pCloudCC_PSYNC_HAS_BUILTIN.zip

@blackjackshellac blackjackshellac mentioned this pull request May 6, 2019
@gdzevo
Copy link
Author

gdzevo commented May 8, 2019

Thank you @blackjackshellac.
Added your diff to separate commit.

@gdzevo gdzevo force-pushed the add-tfa-support branch from 6acb461 to 5f1479a Compare May 8, 2019 21:45
@nikosdion
Copy link

I have tested this PR successfully. Thank you, @gdzevo!

@jaeichel
Copy link

Thank you @gdzevo - rebased and confirmed that your changes still work.

@pcloudcom can you please review, comment, and merge?

@gdzevo
Copy link
Author

gdzevo commented Apr 24, 2022

Closed in favor of rebased PR #163

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants