wltjr (William L Thomson Jr)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Apr 4 2017, 5:23 PM (88 w, 1 d)
Availability
Available

Recent Activity

Jun 1 2018

wltjr updated the task description for T4005: Travis CI and Sonar Scan for Pinentry.
Jun 1 2018, 7:37 PM · pinentry
wltjr created T4005: Travis CI and Sonar Scan for Pinentry in the S1 Public space.
Jun 1 2018, 7:36 PM · pinentry

May 30 2018

wltjr added a comment to T2905: EFL-based pinentry.

@gouttegd Thank you very much!

May 30 2018, 3:32 PM · pinentry, Feature Request

Feb 6 2018

wltjr added a comment to T2905: EFL-based pinentry.

No clue what their problem is, I have a few projects scanned by Coverity. Most are forks that I took over, but one is not really. Not sure why they took such issues here.

Feb 6 2018, 4:46 PM · pinentry, Feature Request

Feb 5 2018

wltjr added a comment to T2905: EFL-based pinentry.

After fighting with Coverity over a fork of pinentry that has EFL. I setup to have Coverity scan. Which found some like 22 defects. Coverity unable to identify that I have any affiliation, after I spent/wasted hours getting a build to upload to Coverity to scan. Just to fight with some unhelpful person basically standing in the way of FOSS project, a wonderful Mel Llaguno. Decided for security reasons I be denied ability to use Coverity to scan pinentry for defects, even in the EFL interface I made and am the author of. Which also means I cannot fix other issues with pinentry or aide further in development....

Feb 5 2018, 11:39 PM · pinentry, Feature Request

Jan 24 2018

wltjr added a comment to T2905: EFL-based pinentry.

Your welcome, I can remake another unified patch if need be. I was starting to prepare things to be a stand alone fork. Did an initial .travis.yml file, and initial stuff for Coverity. Though never did get a build uploaded to Coverity. Not sure if you have ever run pinentry through Coverity or other GnuPG stuff, may be a good idea just to see if it catches anything.

Jan 24 2018, 7:35 PM · pinentry, Feature Request

Jan 23 2018

wltjr added a comment to T2905: EFL-based pinentry.

@werner no problem with re-opening. I closed as it seemed it was not of interest or wanted. I wasn't get any responses like asking why it was left out of 1.1.0 release. To my knowledge other than preferences of GnuPG devs, changes to suit your needs, grabbing, libsecret, etc. It should be good to go without any issues. Thus I was waiting next release, assuming it was already committed . May have confused it with some other PR that was committed. But there should not be any outstanding issues preventing it from inclusion. If there are it was never relayed to me. It should be ready for inclusion, less any requested changes.

Jan 23 2018, 7:26 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

@werner no clue, I thought it was merged in at some point. I could have sworn something happened there. I went on advising others like the TQT interface assuming EFL was already added. I was shocked it was not when release came out and no explanation as to why it was excluded.

Jan 23 2018, 7:20 PM · pinentry, Feature Request

Jan 18 2018

wltjr added a comment to T2905: EFL-based pinentry.

Proceeding with a fork, and likely will remove other interfaces and just maintain another version of pinentry for EFL. Maybe renamed to pinentry-efl, and only have that and tty and curses interfaces in addition to EFL.

Jan 18 2018, 8:13 PM · pinentry, Feature Request

Dec 7 2017

wltjr closed T2905: EFL-based pinentry as Wontfix.

Moving on, I will just look to make a stand along project for efl-pinentry interface. I withdraw my previous submission. Welcome to resume and move forward with Mike Blumenkrantz version. Thanks!

Dec 7 2017, 7:38 PM · pinentry, Feature Request

Dec 3 2017

wltjr added a comment to T2905: EFL-based pinentry.

Not sure this should remain open. Months later a release was done excluding this. Originally mentioned on list in October 2016. Over a year later still not included. Very discouraging. I guess I can just see about having this external for myself. Shocking that FLTK and QTK see more usage than EFL which is part of Tizen OS. Clearly issues with either me, or EFL. Some reason it was excluded and being ignored. Seems nothing I can do either way. Oh well, I did all I could for months. On a very small contribution...

Dec 3 2017, 8:57 PM · pinentry, Feature Request
wltjr added a comment to T3279: Release pinentry 1.1.0.

I could have sworn a patch/pr was merged into repo or something. It seems it was not. Guess I must be mistaking it for some other contribution. Guess I will give up on trying to get EFL into next pinentry release. Which may take another year or so. Despite the fact I have been using it daily for many months now. Oh well.

Dec 3 2017, 8:42 PM · pinentry
wltjr added a comment to T3279: Release pinentry 1.1.0.

Why without it was already committed to repo? Is there some problem I
am not aware of?

Dec 3 2017, 7:28 PM · pinentry
wltjr added a comment to T3279: Release pinentry 1.1.0.
Dec 3 2017, 7:23 PM · pinentry

Oct 13 2017

wltjr added a comment to T3279: Release pinentry 1.1.0.

The --grab/--nograb to my knowledge has nothing to do with the handling of the DISPLAY variable. Which is not used by Wayland. I believe Wayland uses WAYLAND_DISPLAY. @4tmuelle you may want to open a new task on the DISPLAY variable handling.

Oct 13 2017, 7:23 PM · pinentry

Sep 14 2017

wltjr added a comment to T3279: Release pinentry 1.1.0.

Any update on this? Ready to do a pinentry release?

Sep 14 2017, 8:25 PM · pinentry

Aug 6 2017

wltjr added a comment to T3279: Release pinentry 1.1.0.

Password quality evaluation is actually performed by the GnuPG Agent, not by the pinentry programs

Aug 6 2017, 9:43 PM · pinentry

Aug 3 2017

wltjr added a comment to T3279: Release pinentry 1.1.0.

Grabbing the keyboard is an important X feature and not related to gtk etc.

Aug 3 2017, 9:56 PM · pinentry
wltjr added a comment to T3279: Release pinentry 1.1.0.

Hilarious! Not sure if its best to re-comment or edit. Seems phab has edit ability so I tend to do that... I need to slow down and re-read before setting sail :)

Aug 3 2017, 6:41 PM · pinentry
wltjr added a comment to T3279: Release pinentry 1.1.0.

Just corrected :)

Aug 3 2017, 6:38 PM · pinentry
wltjr added a comment to T3279: Release pinentry 1.1.0.

I think a decision should be made about grabbing. Since the gtk pinentry interface seems to be the only one that does grabbing. I need to confirm again, but last I checked, as commented on list, Wayland lacks any grabbing specification or ability. Making this a X only feature of GTK. It seems the patches 1 2 for this are for xwayland, not regular wayland stuff. Which seems to be why bugs like this one on RedHat are open.

Aug 3 2017, 6:36 PM · pinentry

Jun 28 2017

wltjr added a comment to T2905: EFL-based pinentry.
In T2905#99236, @justus wrote:

There is nothing to fix in the way the underlying algorithm communicates its value to the frontend. Negative values mean red, positive values green. After that, you have to normalize that to 0...100.

Jun 28 2017, 3:04 PM · pinentry, Feature Request
wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Added hack to negate negative that move progress bar 10% per 1 char. Fixed issue with new line characters not being handled correctly.

Jun 28 2017, 2:56 PM · pinentry

Jun 27 2017

wltjr added a comment to T2905: EFL-based pinentry.

With all that said, if someone could let me know how you want me to proceed, 2 options.

Jun 27 2017, 7:04 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Looking further into this, pinentry_inq_quality can return a value in the range of -100 to 100. Thus getting -10 from pinentry_inq_quality seems quite normal. Which explains why each are doing <0, since the value can be less than zero, negative quality.

Jun 27 2017, 12:26 AM · pinentry, Feature Request

Jun 26 2017

wltjr added a comment to T2905: EFL-based pinentry.

I am pretty sure I understand it clearly. If I add those two lines it makes the EFL version function like the others. Without it does not. I just debugged this on first character entered, pinentry_inq_quality returns -10. Which again negating -10 becomes 10, and thus the first character gets you 10%, and continues from there.

Jun 26 2017, 10:25 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Said another way, the only thing that should make the progress bar move in any is the "quality" value. The use of percent for the value is the hack. Because the quality value cannot be used. They grab the percent value and increment based on number of characters.

Jun 26 2017, 9:34 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Let me clarify, what all are doing now to make the progress bar move is the following

if(percent<0)
    percent = -percent;

That inverts the value if below zero a negative with 2 negatives become positive. That ends up moving it 10% per 1 character entered. That is the code mine does not have. I have tested it with that code and it functions like all others.

Jun 26 2017, 9:30 PM · pinentry, Feature Request
wltjr added a comment to T2103: Improve the pinentry password quality indication.

I agree with @dkg, and something should be done to address this one way or another. It is pretty misleading.

Jun 26 2017, 8:46 PM · gnupg (gpg23), Feature Request
wltjr added a comment to T2905: EFL-based pinentry.
In T2905#99092, @dkg wrote:

T2103 is the right place to discuss the password quality algorithm, not here.

Jun 26 2017, 8:42 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.
In T2905#98804, @neal wrote:

The password quality algorithm is a joke and is probably more dangerous than helpful. (Try entering the password 12345678...) AIUI, it was added because a client had a specific requirement. I'd prefer that we either fix the algorithm (complicated and depends on the user's threat model) or we deprecate the quality bar.

Jun 26 2017, 6:03 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Just confirmed it is this that causes it to move 10% per char, and is wrong IMHO

if(percent<0)
    percent = -percent;
Jun 26 2017, 5:56 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

I have also noticed that there is a line return after "to" before protect. Which explains why those words run together on the EFL version. I will have to see about replacing the new line characters with something that works for EFL. It does not support new line characters in labels.

Jun 26 2017, 5:42 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

This seems so wrong... entering 1's and a's This would fail a lot of sites that require minimum stuff on passwords like upper/lower, number, special character, etc. This makes NO sense for a quality meter to say junk is quality. Think others have hacked around this. It is not correct.

Jun 26 2017, 5:17 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Even with that being said I see no difference here

Jun 26 2017, 4:38 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Ok I just tested this out and BOTH GTK and QT are messed up. Maybe the others as well. I have to check FLTK and not sure about ncurcses/tty. I simply typed 10 characters and I got 100%. It did not matter what those 10 characters were at all. Like 10 of the same character. That is not correct!!!! That is not saying what I typed was of any quality. This functionality is completely jacked in all. No wonder my version is having issue. Seems others hacked around this broken function and eliminated the entire purposes of qualifying an entry.....

Jun 26 2017, 4:14 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Wait, you said Debian maybe patching YOUR code to fix an issue? Maybe get the patch and apply to pinentry and correct the issue werner found in pinentry. Rather than going off on me for something I have zero control over.

Jun 26 2017, 3:58 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

You do not care about comments in T2905#99021? So werner is completely incorrect there? He stated something is wrong and GTK is effected as well...

Jun 26 2017, 3:57 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

FYI I use this daily as I have been since my first submission. Gentoo ebuild I made which uses the patch. Why would I make that if I am not using? Every commit I make is GPG signed.

Jun 26 2017, 3:55 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.
In T2905#99071, @justus wrote:

No I wont. I'm constantly testing your code. Please read my feedback. I'm growing a bit impatient with you, because I feel like you are developing a piece of software that you are not using, because as soon as I test it I instantly find problems with it.

This is even more worryingly because you actually have multiple pinentries to compare with. The gtk version clearly behaves very differently wrt the quality bar. You need to fix this.

Please re-read my feedback. For example, if you enter "1234567", the gtk one says in red 70%, whereas yours says 0%.

Jun 26 2017, 3:45 PM · pinentry, Feature Request

Jun 24 2017

wltjr added a comment to T2905: EFL-based pinentry.

I have updated the patch in D426, direct link to it on Github, to address the compiler warning from comment T2905#98802 .

Jun 24 2017, 1:29 AM · pinentry, Feature Request
wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Fixed issue with ok_len

../../efl/pinentry-efl.c:493:7: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
       int ok_len = ELM_SCALE_SIZE(strlen(txt) * (PADDING * 1.5));
Jun 24 2017, 1:26 AM · pinentry

Jun 23 2017

wltjr added a comment to T2905: EFL-based pinentry.

ok so I just need to fix the compiler warning and we should be good to go. Was there anything else I needed to address?

Jun 23 2017, 6:28 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

I just tested this out. It seems to be based on what you enter and what is returned from Assuan/Pinentry. If I enter, 2 spaces, then a 1, and repeat that pattern. By the 6th space, you get 20%, and from there it increments by 10% or so to 100% as you continue to enter space space 1,

Jun 23 2017, 6:13 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Why I was saying maybe my math is off or something. I am doing basically the same. Should be the same code. I calculate the percent exactly as they do for GTK. I also set the value the same. Maybe something I am not doing correctly in EFL.

Jun 23 2017, 5:22 PM · pinentry, Feature Request

Jun 21 2017

wltjr added a comment to T2905: EFL-based pinentry.

The quality bar should be working, please try typing in more characters till it does something. It should at some point.

Jun 21 2017, 5:41 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

In my tests it worked, you just have to type a decent amount to get it to kick in. It seems to accelerate really quick, like it jumps from 0 to 60% and then 100%, not really smooth from 0 - 100, as intended. But I think that is due the quality value returned from Pinentry.

Jun 21 2017, 5:35 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

How is that icon by the way? Like the key better than the lock/shopping bag? :) The icons will change based on the users selected icon set.

Jun 21 2017, 4:53 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

I will fix that warning, I should have caught that, I do no think I am using that compiler flag/option.

Jun 21 2017, 4:31 PM · pinentry, Feature Request

Jun 8 2017

wltjr added a comment to T2905: EFL-based pinentry.

I updated the patch, fixed all issues mentioned and a couple others I noticed. Things not being centered vertically labels/entries, and ok not being fired on pressing enter on entry, or confirm when present. That should fix all outstanding issues.

Jun 8 2017, 6:05 PM · pinentry, Feature Request
wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Various fixes

  • Fixed confirm passphrase intial text, was label text...
  • Fixed button length with long text
  • Fire ok on pressing enter in entry or confirm if present
  • Call focus on entry last, attempt to fix focus
  • Vertically center labels with entries (minor were aligned to top vs center/middle)
Jun 8 2017, 6:01 PM · pinentry
wltjr added a comment to T2905: EFL-based pinentry.
In T2905#98415, @justus wrote:

So are you also saying that I should better not use e17 because its focus handling is so fubar that it does not focus the pinentry when it pops up?

Jun 8 2017, 4:43 PM · pinentry, Feature Request

Jun 7 2017

wltjr added a comment to T2905: EFL-based pinentry.

@justus Can you tell me how you got the two passwords with extra text and the long button text? I can replicate the long button text via cli. Not sure about the two passwords and extra unwanted characters. I would like to be able to replicate as you did. Thank you!

Jun 7 2017, 6:08 PM · pinentry, Feature Request

May 31 2017

wltjr added a comment to T2905: EFL-based pinentry.

I got your point, I was saying do not have a chat client or program that would create pop ups and grab focus away. Its a highly debatable and personal preference type of thing. I have run into such already.

May 31 2017, 3:58 PM · pinentry, Feature Request

May 25 2017

wltjr added a comment to T2905: EFL-based pinentry.

Updated the patch should be good to go now

May 25 2017, 4:46 AM · pinentry, Feature Request
wltjr updated the diff for D426: Initial patch for EFL based pinentry.
  • Fixed issues with underscores in button text
  • Fixed issues with warnings
  • Added news item
  • Fixed issue with show/hide changing size
  • Replaced icon, fixed missing button icons
May 25 2017, 4:46 AM · pinentry

May 24 2017

wltjr added a comment to T2905: EFL-based pinentry.

Ok I can add the keyboard/mouse grab stuff. I have the code already. I get your point, mine is the opposite of yours. I would say don't launch something if your typing in your pin or about to :)

May 24 2017, 5:34 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

I will see about removing the underscores now that I understand their meaning. I am not sure if EFL has any means to interpret such at this time. I will look into it and address either way. Thank you for that information!

May 24 2017, 4:28 PM · pinentry, Feature Request

May 23 2017

wltjr added a comment to T2905: EFL-based pinentry.

Forgot EFL version...

May 23 2017, 5:51 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Ok you should be good to go now. There are 2 issues I am aware of.

May 23 2017, 5:47 PM · pinentry, Feature Request
wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Updated diff, swapped out legacy api function

May 23 2017, 5:44 PM · pinentry
wltjr added a comment to T2905: EFL-based pinentry.

Very sorry! I already fixed that. I just had not updated the patch. This one is updated
https://github.com/Obsidian-StudiosInc/pinentry/commit/0fb3104c3ab27112aad70668c5828f9d435e10d4.patch

May 23 2017, 5:43 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

What version of the patch or EFL?

May 23 2017, 4:44 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

I sent the DCO per request.

May 23 2017, 3:38 PM · pinentry, Feature Request

May 12 2017

wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Switched to dialog from window. Relocated call to center window. Which may not be correctly centered till release of EFL 1.19.1 due to a bug in EFL itself. I thought was a bug in my code. Thus previous patch.

May 12 2017, 5:50 PM · pinentry
wltjr added a comment to T2905: EFL-based pinentry.

If the dialog's show a bit off centered. The center of the screen being top/left of dialog. Which makes it offset to the bottom right. That is a bug in EFL (T5481) that is fixed and should be in EFL 0.19.1. Not anything related to this code though I did try to address in this code.

May 12 2017, 5:17 PM · pinentry, Feature Request
wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Minor changes, switched to dialog. Modified code to fix centering issues that may be a bug in EFL itself.

May 12 2017, 3:48 AM · pinentry

Apr 28 2017

wltjr added a comment to T2905: EFL-based pinentry.

I have updated the code and patch. It is ready for review, modification, and ideally inclusion

Apr 28 2017, 11:50 PM · pinentry, Feature Request
wltjr added a reviewer for D426: Initial patch for EFL based pinentry: pinentry.
Apr 28 2017, 11:49 PM · pinentry
wltjr updated the diff for D426: Initial patch for EFL based pinentry.

Cleaned up code format some. I fixed issue when display is not present and fallback to ncurses. This should be good to go now. Pending review, other issues, reformatting etc.

Apr 28 2017, 11:48 PM · pinentry

Apr 24 2017

wltjr added a comment to T2905: EFL-based pinentry.

I have noticed some issues, minor code fixes, and a major issue with failing to fall back to tty/ncurses interface when a GUI is not available. I will make the changes, cleanup the code format, etc and re-attach patch.

Apr 24 2017, 5:31 PM · pinentry, Feature Request

Apr 19 2017

wltjr added a comment to T2905: EFL-based pinentry.

The underscores on the Cancel and OK button come from Pinentry. Not sure if I am not handling Locale correctly, or something along those lines. The other stuff does not have it, and setting text the same way.

Apr 19 2017, 4:48 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

Sure thing, and it is "semi" animated. If you fail to enter a correct pin/passphrase. The error message is animated, it will slowly move back and forth from left to right. I can provide further screenshots of its various options, confirm, quality, double entry, etc. The quality bar changes color from red to green, with every shade in between based on quality, 0 red, 100 green, in between other colors/shades between the two.

Apr 19 2017, 4:46 PM · pinentry, Feature Request

Apr 18 2017

wltjr added a comment to T2905: EFL-based pinentry.

I changed it back to EFL from Enlightenment. Enlightenment is a desktop/application coded using the EFL. Like Gnome is coded in GTK. This does not require Enlightenment at all. Just the EFL. Which can be used in GTK/Gnome, or KDE/QT based environments just the same. Also Tizen, etc. Thus I would not call it Enlightenment anywhere, as it really has no relation.

Apr 18 2017, 10:30 PM · pinentry, Feature Request
wltjr renamed T2905: EFL-based pinentry from Enlightment-based pinentry to EFL-based pinentry.
Apr 18 2017, 10:28 PM · pinentry, Feature Request
wltjr added a comment to T2905: EFL-based pinentry.

I created the requested differential. Please let me know what I need to do for inclusion. Thanks!

Apr 18 2017, 10:27 PM · pinentry, Feature Request
wltjr created D426: Initial patch for EFL based pinentry.
Apr 18 2017, 10:26 PM · pinentry

Apr 14 2017

wltjr added a comment to T2905: EFL-based pinentry.

Ok I will do that soon as I am finished with refinements and it is ready for re-submission. Thanks!

Apr 14 2017, 3:01 PM · pinentry, Feature Request

Apr 13 2017

wltjr added a comment to T2905: EFL-based pinentry.

I got the patch from Mike. It does need some refinements. I will work on the modifications and re-submit. Do you accept PRs on Github? Or should I attach here? Or send to mailing list? Thanks!

Apr 13 2017, 5:18 PM · pinentry, Feature Request

Apr 4 2017

wltjr added a comment to T2905: EFL-based pinentry.

I am interested in this, and can look into any needed polishing. Can someone please attach the patches to this bug or a link to where they can be downloaded. Unfortunately mailing list archives strip attachments. I spoke with Mike in IRC who informed me he made a EFL port of pinentry. I was working on that, and still have my code going in case I cannot get a copy of Mikes work. I am willing to do what ever is necessary to get this into pinentry sooner than later. Though I understand it can take some time. Hopefully not the 1yr it took the FLTK based pinentry to be added.

Apr 4 2017, 7:20 PM · pinentry, Feature Request