search

aedmonds / Magento-Two-factor-Authentication

Should be time-based or counter-based (HOTP/TOTP), and support the Google Authenticator mobile app.
https://github.com/orgs/magento-hackathon
7 stars 1 forks source link

Can we use something better instead of serialized strings? #9

Open svenvarkel opened 10 years ago

svenvarkel commented 10 years ago

image

jonpday commented 10 years ago

the serialized array is necessary to support the dynamic grid control in the System>Config>Admin>Security section. @willtran can you give you more info on that

svenvarkel commented 10 years ago

I mean more like can we add a list of resources as xml elements there? I know it may add additional overhead for parsing it. Smth like:

<twofactorauth_protected_resources>
    <resource>admin/sales/order</resource>
    <resource>admin/system</resource>
    <resource>and/so/on...</resource>
</twofactorauth_protected_resources>

However, this is low priority issue