- Issue created by @DamienMcKenna
- πΊπΈUnited States DamienMcKenna NH, USA
Looking at the codebase there are no migration plugins, so that means we'll need some work done:
* Source plugin for the Protected Node data.
* (probably) Process plugin to store or convert the data to the correct format.
* Migration YML file for mapping the source.
* Migration state file. - π«π·France Grimreaper France π«π·
Hi,
@DamienMcKenna, thanks to open this issue.
Currently there is only an example module for people who wants/can do custom migrations: https://git.drupalcode.org/project/entity_access_password/-/tree/1.0.x/t...
- Assigned to DamienMcKenna
- πΊπΈUnited States DamienMcKenna NH, USA
Working on this.
While digging around the code I found that Protected Node uses hash('sha256') while EAP uses core's Password service which uses hash('sha512'). So I don't know if this will work :-\
- Issue was unassigned.
- Status changed to Needs work
about 1 year ago 5:47pm 25 August 2023 - πΊπΈUnited States DamienMcKenna NH, USA
My WIP. After realizing that the passwords can't be converted, I don't think there's any benefit to continuing on this?
- π«π·France Grimreaper France π«π·
Hi,
Thanks @DamienMcKenna for your work.
When doing EAP one year and a half ago, I tested if password could be reused as-it-is. And found out that D7 user account passwords worked on D9.
So that's why I made the following change on Protected Node. So with passwords saved from version 8.x-1.8 of Protected Node it works.
- Status changed to Needs review
about 1 year ago 5:27pm 21 September 2023 - last update
about 1 year ago 5 pass - πΊπΈUnited States DamienMcKenna NH, USA
I forgot to include all the "use" statements in the module, so the migration review page would blow up. Or in fact probably everything would blow up. Oops.
- Status changed to Needs work
about 1 year ago 5:30pm 21 September 2023 - πΊπΈUnited States DamienMcKenna NH, USA
Next steps:
* Migrate the password as-is, it'll either work or at least provide an indication that there *was* a password on the old site that then needs to be re-entered.
* Test =)
* Maybe write some test coverage. - Assigned to DamienMcKenna