. '''To Software [[Software|Software]]''' - '''To Software-Assessment [[Software/Assessment|Software/Assessment]]''' - '''To [[Software/Assessment/20120710-S-A-MiniTOP|previous meeting]]''' - '''To [[Software/Assessment/20120724-S-A-MiniTOP|next meeting]]''' ---- = Minutes of the MiniTOP on the 2012-07-17 = == Setting == The MiniTOP will be held via telco 22:00 CEST Attendees: Marcus, Uli, Benny, dirk, Michael, (David via irc) == Topics == (skip to [[#AGENDA|agenda]]) Action items from last meeting '''[[Software/Assessment/ActionItems|Meeting Action Items]]''' <> <> == Agenda == ## last full agenda https://wiki.cacert.org/Software/Assessment/20120228-S-A-MiniTOP === 1. Preface === 1. Cebit brainstorming * dirk: request for events report * (2012-03-27) Marcus awaiting translation from Marc * (2012-06-19) Marcus: translation received, will send within the next upcoming days * (2012-06-26) Marcus: not yet finished * 2nd draft finished * Sat report missing 1. Bennys c.o address * wip === 2. 2nd review of about 5 patches === ||<#ff8080> '''Software-Assessors task''' || 1. [[https://bugs.cacert.org/view.php?id=789|bug #789]] OA edit domain fix, Editing domain for organisations does not work<
>new update 2011-09-26<
>2 tests, needs 2nd review, deploy<
>more fixes, more testing * 2nd review of 1 patch * Michael cannot do, needs doing by dirk (or other Software-Assessor, who else?) * [[https://bugs.cacert.org/view.php?id=789|bug #789]] reviewed: 2012-07-10 * what is /pages/account/29.php for? edit org domain * (pc vm crashed) 1. [[https://bugs.cacert.org/view.php?id=978|bug #978]] bug 978 (weak keys) (bug 918) * invalid key format, no regular error message, something wrong, error code # identified * debugging infos from user + infos from critical team with error code #, was spkac routine * one test done 2011-12-17 by JensK * uli, marcus: more tests: certs routine, weak keys (small keys test), relates to [[https://bugs.cacert.org/view.php?id=540|bug#540]] tests * (week 7) 1. [[https://bugs.cacert.org/view.php?id=540|bug#540]] No key usage attribute in cacert org certs anymore? * also: [[https://bugs.cacert.org/view.php?id=905|bug#905]] * Policy group discussion - Extended key usage -> [[PolicyDecisions#p20111113|p20111113]], motion CARRIED * deployment 1. prepare fixes -> Michael to prepare diffs, against svn 1. sending to testserver 1. transfer to critical system * (2011-12-13) approx 2 weeks to write the fix, approx 2 months to go * Michael did transfer the patch to testserver * signer code update * changes against svn * uli, to add to tester portal, done * uli to inform testers about new tests * test report from kenneth to transfer to report (email from 2011-12-25) * Michael: where to find the report from kenneth? link? * NEO has added the report (written to private dl) * who has adobe 8 for testing? * magu has, please test * next: needs testing (week 6) * uli, marcus: needs full cert create tests * uli (2012-01-25): sent notification to software testers * awaiting testing ... problem FULL test, including all possible variations with certs creation * also to report under [[https://bugs.cacert.org/view.php?id=978|bug #978]] bug 978 (weak keys) (bug 918) * Testers: test all certs veriations, functions * dirk 2nd review of patches, reviewed 2012-07-10 * [[https://bugs.cacert.org/view.php?id=540|bug #540]] * diff line 23ff unclear, what does section ($root==2) mean? * also unclear: else section $CRLUrl="http://crl.cacert.org/root${root}.crl"; * skipped /!\ 1. [[https://bugs.cacert.org/view.php?id=1024|bug #1024]] reviewed 2012-07-10 * server.pl, too much changes to review in a working session, skipped /!\ || neo || [[https://bugs.cacert.org/view.php?id=1024|bug #1024]] Assurer flag is not set correctly on updatesort.php run || tested by 4, ok || 2 {0} || || Michael || [[https://bugs.cacert.org/view.php?id=540|bug #540]] || p20111113 CPS #7.1.2 "Certificate Extensions" adjustments - testing<
>uli, marcus: needs full cert create tests<
>duplicate report to bug#978 || 3 {0} || || inopiae || [[https://bugs.cacert.org/view.php?id=981|bug #981]] OA overview (dupe of [[https://bugs.cacert.org/view.php?id=943|bug #943]]) || New layout of view for Organisation Administrators in account/id35 || 4 {0} || || neo || [[https://bugs.cacert.org/view.php?id=978|bug #978]] Invalid SPKAC requests are not properly validated || recheck full certs signing procedures<
>duplicate report to bug#540 || 5 {0} || || uli, ted || [[https://bugs.cacert.org/view.php?id=789|bug #789]] OA edit domain fix || Editing domain for organisations does not work<
>new update 2011-09-26<
>2 tests, needs 2nd review, deploy<
>more fixes, more testing || 6 {0} || === 3. bug #1023 Testing (6.php) === 1. Thawte points removal, final step * last patch transfered to production system 2012-05-30 1. what are the next steps for thawte points revoke? * points settings codes eg 50 pts open gpg/pgp, which certs avail by how many pts * 15.php needs rename to 10.php * cannot move forward without dirk === 4. Marcus Bugs list === * see also [[Software/BugsOverview]] * [[https://bugs.cacert.org/view.php?id=1023|bug#1023]] related * [[https://bugs.cacert.org/view.php?id=583|bug#583]] "Assure Somebody" allows future assurance dates * [[https://bugs.cacert.org/view.php?id=648|bug#648]] send message from Assurer to Member * [[https://bugs.cacert.org/view.php?id=802|bug#802]] Name parts should be designated in assurance form * [[https://bugs.cacert.org/view.php?id=870|bug#870]] My Details - My Points show bugus time stamp * [[https://bugs.cacert.org/view.php?id=914|bug#914]] Information about Practice on Name while entering an Assurance * [[https://bugs.cacert.org/view.php?id=930|bug#930]] types wrong points in "Assure Someone" form * [[https://bugs.cacert.org/view.php?id=931|bug#931]] Date of assurance in future don't throw any exception * [[https://bugs.cacert.org/view.php?id=998|bug#998]] When entering an assurance in the WoT one line of the form the suffix is given in another line the suffix is missing. * [[https://bugs.cacert.org/view.php?id=1000|bug#1000]] Entering an assurance into the system after searching for an assurer causes a pre-filled location field * Others * [[https://bugs.cacert.org/view.php?id=118|bug#118]] Secure TTP Form upload - outdated, conflicts with new procedure, closed * [[https://bugs.cacert.org/view.php?id=428|bug#428]] Reminder language-drop-down-box doesn't keep "English" if you choose it again - cannot be reproduced, tested by 2, closed * [[https://bugs.cacert.org/view.php?id=489|bug#489]] Pb on rewarding 2 points for an assurance * [[https://bugs.cacert.org/view.php?id=567|bug#567]] case sensitive email: tested by 2, cannot be confirmed, closed * [[https://bugs.cacert.org/view.php?id=767|bug#767]] Single-quotes escaped in Web-of-Trust contact form. * info pages to wiki pages * starting [[https://bugs.cacert.org/view.php?id=671|bug #671]]. there still exist a bug# [[https://bugs.cacert.org/view.php?id=740|bug #740]] (How to become an assurer is missleading) * [[https://bugs.cacert.org/view.php?id=491|bug #491]] "Please allow usage of "secondary" emails user ids." - proposes: Close with rejected {{{ * username/password half of the combination is known to potential attacker * login prevents login to several email addresses * acceptance to several email addresses is prevented * no notification if primary email address has been changed * note regarding Policy Group * dirk: proposal: response email address exists, but isn't primary email ? * create new account results in "email address exists" * what is a proper response? * requestor has to be an assurer for assure someone * neo: for registration process chaptcha required * no good solution * for assurance only primary, for all other services allow also secondary addresses * search needs enhancement: search not only primary, also secondary }}} * [[https://bugs.cacert.org/view.php?id=571|bug #571]] "need for email addresses (or link) in admin console" - proposes: Closed with solved by other bug fix {{{ * primary and secondary email addresses are shown in admin console }}} * [[https://bugs.cacert.org/view.php?id=591|bug #591]] "CPS has to be improved for audit." - proposes: Closed {{{ * CPS is a working revision also DRAFT revision included * relates to policy repository bug# final place finding }}} * addtl. groups: a. OA a. CCA rollout a. TTP === 5. Benny reviews === 1. [[https://bugs.cacert.org/view.php?id=1025|bug #1025]] "Domain Dispute strange behaviour / Domain Dispute issue", checked * wrong description, problem removing domains, bugfix solves this problem * async removal of certs by signer * needs review and testing * inopiae will try testing on upcoming weekend * to test: email- and domain dispute 1. [[https://bugs.cacert.org/view.php?id=922|bug #922]] "CAcert application code problem causing missing 'certificate about to expire' messages", checked * patch seems to be ok * white spaces cleanup * includes/account.php var $id shall be fixed within recursion, new [[https://bugs.cacert.org/view.php?id=1078|bug #1078]] * 2 tests initiated by inopiae and u60 * principle ok, but very confusing * test reports Marcus: * discussions, Marcus got 71 or 72 notifications * Neo: default 5 notifications: 45d, 30d, 15d, 3d, 1d * [[https://bugs.cacert.org/view.php?id=922|bug #922]] test report / review * one test account, 1 client cert, 1 server cert, received 105 (1) reminders (!!!) * 15 reminders checked, 1 for client cert, 14 for server cert (!!!) * needs further inspection 1. [[https://bugs.cacert.org/view.php?id=1019|bug #1019]] "Contact form does not work when logged in" * Michael: rework contact form * usability: 1 form, option box with public/support delivery, default support * current form 1: public, form 2: private * spam prevention via java, on disabled java the mail is marked [possible spam] * mass mailing possible if adding multiple emails separated by commas * account.php - email address from sender, no address validation, several other places it passes address validation * neo: why not use primary email address? * works only if logged-in * index?id=11 has also been changed * url was hardcoded * account.php?id=14 * sendmail() routine in includes/mysql.php 1. patches 2nd review, Benny to do pre-view || neo || [[https://bugs.cacert.org/view.php?id=1024|bug #1024]] Assurer flag is not set correctly on updatesort.php run || tested by 4, ok || 2 {0} || || Michael || [[https://bugs.cacert.org/view.php?id=540|bug #540]] || p20111113 CPS #7.1.2 "Certificate Extensions" adjustments - testing<
>uli, marcus: needs full cert create tests<
>duplicate report to bug#978 || 3 {0} || || inopiae || [[https://bugs.cacert.org/view.php?id=981|bug #981]] OA overview (dupe of [[https://bugs.cacert.org/view.php?id=943|bug #943]]) || New layout of view for Organisation Administrators in account/id35 || 4 {0} || || neo || [[https://bugs.cacert.org/view.php?id=978|bug #978]] Invalid SPKAC requests are not properly validated || recheck full certs signing procedures<
>duplicate report to bug#540 || 5 {0} || || uli, ted || [[https://bugs.cacert.org/view.php?id=789|bug #789]] OA edit domain fix || Editing domain for organisations does not work<
>new update 2011-09-26<
>2 tests, needs 2nd review, deploy || 6 {0} || * for #540 uli has sent a short summary to dirk === 6. New SA candidates and Coders === 1. ABC Benny - possible Itzehoe (2012-09-14), mrmcd (2012-09-08) or other events before 2012-08-10 - 2012-08-11 BarCamp kiel * [[Arbitrations/a20120703.1|ABC Benny]] 1. Whats with ABC over archaios? 1. How to find coders? Experiences from the Gentoo project * [[http://redmonk.com/dberkholz/2012/07/10/how-to-recruit-open-source-contributors/]] * [[http://www.slideshare.net/dberkholz/lessons-on-recruiting-open-source-contributors-from-the-google-summer-of-code]] === 7. English Translation Problems === * how to handle typing error in web phrase [[Software/TranslationMisspelling]] . "Can't continue with certificaterequest." in ../includes/account.php:341 ../includes/account.php:1482 === 8. Long Term Projects === 1. NEO: "BlackJack" * info from NEO: "BlackJack" moved forward 1. Marek's sql class project: * is working on charset replacement 1. api project, Carsten continues with portal project not waiting for vendor-api to be delivered * potential candidates for development 1. Marek's sql class proposal * needs probably db upgrades * needs addtl. indices * needs testing 1. archaios * builds daemon as unpreviliged user * vendor-api delayed * no coders * other projects * related to sql class project * portal project continues with a workaround, needs an assurer * arbitration case on locations database orders outsourcing of find-an-assurer asap * with portal function, update of data is possible vs. update of data on critical system is difficult (keep data current for assurers) === 9. next meeting === * Tuesday, July 24, 2012 22:00 CEST == Minutes == 1. Preface 1. Bug Testing / Reporting bug #922 difficult * Marcus writes a tool to collect Email infors from TMS 1. Bennys reviews * 5 patches reviewed * 3 simple, bugs 540 (mostly check policy text), 789, 981 * 2 with some difficultys, complexest one: 1024 1. Bennys c.o address * is active, finished 1. ABC Benny, no fixed date set yet 1. NEO: default pem coded signed key output, chrome expects der 1. How TMS email works: NEO: TMS connects via imap and collects and displays mails 1. dirk review bug 540 * gitdiff origin/release..origin/bug540 * dirk 2nd review of patches, reviewed 2012-07-10 * [[https://bugs.cacert.org/view.php?id=540|bug #540]] * diff line 23ff unclear, what does section ($root==2) mean? * class 3 server (in signer), comm module, each root client certs, there is a config #, root=0 class1, root=1 class3, root=2 newroots project class3s.crl (from 2008 new roots project) * root=2 not avail on current system * also unclear: else section $CRLUrl="http://crl.cacert.org/root${root}.crl"; * 5 root vars defined (in client.pl) * crl for other keys, not class1, class3. all other keys not active on current system * server.pl: root.crl, class3, class3s, for further still unused keys * related policy decision: [[https://wiki.cacert.org/PolicyDecisions#p20111113]] * review ok * config files not reviewed, 2nd review not finished 1. [[https://bugs.cacert.org/view.php?id=1075|bug #1075]] cap form link wrong under pages/wot/6.php || neo || [[https://bugs.cacert.org/view.php?id=1075|bug #1075]] cap form link wrong under pages/wot/6.php || cap link removed, moved to testserver || {0} || * data protection problem to pickup user data before assurance f2f meeting starts * what does assurance process means? assurance "process" starts from request of assuree to an assurer to do an assurance over assuree * problem in ttp process too, to have a view over data before f2f meeting and signed cap is in the hands of an assurer. ttp-admin can request confirmation from ttp-user to access online data * simple patch: remove links * edited by NEO: transfered to testserver 1. dirk review bug 789, OA edit domain fix, Editing domain for organisations does not work * gitdiff origin/release..origin/bug789 * [[https://bugs.cacert.org/view.php?id=789|bug #789]] reviewed: 2012-07-10 * what is /pages/account/29.php for? edit org domain * phone accu breaks 1. NEO: has finished IE patch, [[http://cacert.nhng.de/IEkeygen/keygen.html]] * will prepare a working patch and will transfer to testserver within the next 7 days 1. NEO: git diff origin/release...origin/bug-XXX -> search all differences since last release 1. How to find coders? Experiences from the Gentoo project * [[http://redmonk.com/dberkholz/2012/07/10/how-to-recruit-open-source-contributors/]] * [[http://www.slideshare.net/dberkholz/lessons-on-recruiting-open-source-contributors-from-the-google-summer-of-code]] * use as blueprint for all recruits? * some discussions 1. English Translation Problems * how to handle typing error in web phrase [[Software/TranslationMisspelling]] * "Can't continue with certificaterequest." in ../includes/account.php:341 ../includes/account.php:1482 * create shared bug * probably make part a. and b. a. that is clear, b. that is questionable 1. David's posts in irc 1. there is no checks for \00, er, \0 * the \0 check will be done in signer (-> CommModule client and/or server.pl) * server.pl lines 494+495 * blacklist for domain names for new signer 1. (char) 160 is problematic in various locales, as it appears as whitespace (160 is not a particularly good val either in ISO-8859-1) in certs * todo: doing whitelist of allowable chars * \xA0 is a problem too (at least in Win32/64) 1. subjectAltName is occasionally not checked for problems * file a new bug 1. FF prob with favicon verification * [[http://forums.mozillazine.org/viewtopic.php?t=636366]] * some discussion 1. next meeting * Tuesday, July 24, 2012 22:00 CEST ==== Fixed Action Items since last or within meeting ==== ==== Action Items New ==== Action items: '''[[Software/Assessment/ActionItems|Meeting Action Items]]''' ---- . CategorySoftwareAssessment