<table><tr><td style="">WDavidO created this revision.<br />WDavidO added a reviewer: KDevelop.<br />WDavidO set the repository for this revision to rKDEVELOP KDevelop.<br />WDavidO added a project: KDevelop.<br />Restricted Application added a subscriber: kdevelop-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D3395" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Patch appends local host name and current date time to personal access token generated for kdevelop github plugin so it doesn't fail despite correct credentials when another instance of kdevelop is already authorized on the same github account.</p>

<p>This may also fix bug <a href="https://bugs.kde.org/show_bug.cgi?id=360589" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=360589</a> I don't have windows 8.1 and have never experienced a crash when previous authentications failed.</p>

<p>In addition to appending to the generated token names I created a more verbose failure message which displays the name that was attempted and a link to the github personal access tokens page and a similar information message when authentication succeeds so users are better able to manage the tokens they create. I also updated the VALID_ACCOUNT message to link to the personal access tokens section of github settings (where the created token will be listed) as opposed to the generic applications section.</p>

<p>The message boxs may be a bit overkill but would have saved me a lot of time in the original diagnosis of why tokens couldn't be created with the old method. That being said name collisions should be virtually impossible now so the messages could be slimmed down or omitted. I am reasonably concerned that someone could purge and reinstall kdevelop or something of that nature and have a number of stray access tokens without realising it so I decided to pursue a verbose approach.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Using machines A and B (virtual is acceptable)  both with KDevelop installed and a github account that is authorized for any kdevelop installation.</p>

<p>authorize the github account with kdevelop on machine A and it will work.</p>

<p>Attempt to authorize the same account on machine B and it will not work without this patch. The error will also not give any indication as to why the authorization failed.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKDEVELOP KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3395" rel="noreferrer">https://phabricator.kde.org/D3395</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>providers/ghprovider/ghdialog.cpp<br />
providers/ghprovider/ghdialog.h<br />
providers/ghprovider/ghresource.cpp<br />
providers/ghprovider/ghresource.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>WDavidO, KDevelop<br /><strong>Cc: </strong>kdevelop-devel, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>