Merging agelimits (#2)

* Added support for skipping ageLimits credential

* Make idin credential id configurable for testing

* Added user warning that attributes are going to be issued twice

* Improved text (feedback Bart)

* chore: Specify that the double issuance will end somewhere in 2020, and not necessarily a year after this change.

Gruntfile.js

@@ -10,11 +10,15 @@ module.exports = function (grunt) {
     if ( (typeof(grunt.option("language")) === "undefined") ) {
         console.log("INFO: No language chosen, assuming nl");
     }
+    if ( (typeof(grunt.option("idin_credential_id")) === "undefined") ) {
+        console.log("INFO: No idin_credential_id chosen, assuming pbdf.pbdf.idin");
+    }
 
     var conf = {
         idin_server_url: grunt.option("idin_server_url") || "<IDIN_SERVER_URL>",
         irma_server_url: grunt.option("irma_server_url") || "<IRMA_SERVER_URL>",
         language: grunt.option("language") || "nl",
+        idin_credential_id: grunt.option("idin_credential_id") || "pbdf.pbdf.idin",
     };
     conf.irma_server_url += "/api/v2";
 

src/enroll.html

@@ -41,6 +41,8 @@
                 <tbody></tbody>
             </table>
 
+            <p id="twoCredentialsWarning" style="display: none"><i><%= enroll_two_credentials_warning %></i></p>
+
             <p><%= enroll_clicktoload %></p>
             <button id="enroll" class="btn btn-primary"><%= enroll_load_button %></button>
             <hr />

src/js/iDIN-enroll.js

@@ -15,14 +15,16 @@ getSetupFromJson(function() {
 
     function displayAttributes (creds) {
         $.each(creds, function(i, cred) {
-            if (cred.credential === "pbdf.pbdf.idin"){
+            if (cred.credential === conf.idin_credential_id){
                 $.each(cred.attributes, function(key, value) {
-                    addTableLine($('#idinTable'), strings.hasOwnProperty("attribute_" + key) ? strings["attribute_" + key] : key, value);
+                    if (key.includes("over")) {
+                        addTableLine($('#ageTable'), strings.hasOwnProperty("attribute_" + key) ? strings["attribute_" + key] : key, value);
+                    } else {
+                        addTableLine($('#idinTable'), strings.hasOwnProperty("attribute_" + key) ? strings["attribute_" + key] : key, value);
+                    }
                 });
             } else {
-                $.each(cred.attributes, function(key, value) {
-                    addTableLine($('#ageTable'), strings.hasOwnProperty("attribute_" + key) ? strings["attribute_" + key] : key, value);
-                });
+                $('#twoCredentialsWarning').show();
             }
         });
     }

src/js/iDIN-verify.js

@@ -2,7 +2,7 @@ getSetupFromJson(function() {
     var success_fun = function(data) {
         $("#result_status").html(strings.verify_success);
         $("#result_header").html(strings.verify_result);
-        var bd = jwt_decode(data).attributes["pbdf.pbdf.idin.dateofbirth"];
+        var bd = jwt_decode(data).attributes[conf.idin_credential_id + ".dateofbirth"];
         $("#token-content").html("<b>" + strings.verify_birthdate + "</b> " + bd);
         //TODO: check for birthdate
     }

src/languages/en.json

@@ -12,6 +12,7 @@
 	"enroll_header": "Beschikbare attributen",
 	"enroll_received_attributes": "De volgende attributen zijn beschikbaar gesteld door uw bank:",
 	"enroll_derived_attributes": "De volgende attributen zijn daaruit afgeleid:",
+	"enroll_two_credentials_warning": "In november 2019 is een verandering in de structuur van de iDIN attributen in de IRMA-app doorgevoerd, waarbij leeftijdsgrenzen niet meer apart staan. Om deze verandering soepel te laten verlopen, worden deze leeftijdsgrenzen tijdelijk dubbel uitgegeven: eenmaal als onderdeel van het geheel van iDIN attributen en eenmaal los. In de loop van 2020 zal de uitgifte van de losse leeftijdsgrensattributen definitief worden uitgefaseerd.",
 	"enroll_clicktoload": "Klik de Laad attributen knop om deze attributen in uw IRMA app te laden.",
 	"enroll_load_button": "Laad attributen in uw IRMA app",
 	"done_title": "iDIN Attributen uitgegeven",

src/languages/nl.json

@@ -12,6 +12,7 @@
 	"enroll_header": "Beschikbare attributen",
 	"enroll_received_attributes": "De volgende attributen zijn beschikbaar gesteld door uw bank:",
 	"enroll_derived_attributes": "De volgende attributen zijn daaruit afgeleid:",
+	"enroll_two_credentials_warning": "In november 2019 is een verandering in de structuur van de iDIN attributen in de IRMA-app doorgevoerd, waarbij leeftijdsgrenzen niet meer apart staan. Om deze verandering soepel te laten verlopen, worden deze leeftijdsgrenzen tijdelijk dubbel uitgegeven: eenmaal als onderdeel van het geheel van iDIN attributen en eenmaal los. In de loop van 2020 zal de uitgifte van de losse leeftijdsgrensattributen definitief worden uitgefaseerd.",
 	"enroll_clicktoload": "Klik de Laad attributen knop om deze attributen in uw IRMA app te laden.",
 	"enroll_load_button": "Laad attributen in uw IRMA app",
 	"done_title": "iDIN Attributen uitgegeven",