Change Details

IP Masking will affect lots of our products, features, tools, gadgets, etc. This task is for tracking work to update those that are owned by ~~#Content-Transform-Team~~ #structured_data_engineering, ahead of IP Masking being enabled on WMF sites. See {T326816}, particularly **What will be affected**. A preliminary investigation (T326759) has found that the following may be affected: * MachineVision

IP Masking means that we create a temporary account for any user that's editing without being logged in. More details on how temporary accounts work [[ https://www.mediawiki.org/wiki/Help:Temporary_accounts/How_it_works | here ]], and more details on what to look for when adapting an extension to handle temporary accounts is [[ https://phabricator.wikimedia.org/T326816 | here ]] A quick search for relevant code in the Machine extension is [[ https://codesearch.wmcloud.org/deployed/?q=-%3EisAnon%7C-%3EisRegistered%7Cmw%5C.user%5C.isAnon%7Cmw%5C.user%5C.getId%7Cmw%5C.user%5C.getRegistration&files=&excludeFiles=&repos=mediawiki%2Fextensions%2FMachineVision | here ]] ATM Special:SuggestedTags requires a user to be logged in, which suggests that we ought not allow a temp user to use it ... but we should make sure that there's no login required to use the API (so the ISA tool can continue to work). Probably we'll need to replace `isAnon` with `!isNamed`, make sure that an anon/temp user can't reach the `mw.user.getId` code, find where the code checks if a user is logged in, and then test everything to make sure it still works when logged in as a regular user . To enable temporary accounts in your local dev env, add this to LocalSettings.php ``` $wgAutoCreateTempUser['enabled'] = true; ```

IP Masking will affect lots of our products, features, tools,means that we create a temporary account for any user that's editing without being logged in. gadgets, etc.More details on how temporary accounts work [[ https://www.mediawiki.org/wiki/Help:Temporary_accounts/How_it_works | here ]], This task is for tracking work to update those that are owned by ~~#Content-Transform-Team~~ #structured_data_engineering, ahead of IP Masking being enabled on WMF sites.and more details on what to look for when adapting an extension to handle temporary accounts is [[ https://phabricator.wikimedia.org/T326816 | here ]] See {T326816}A quick search for relevant code in the Machine extension is [[ https://codesearch.wmcloud.org/deployed/?q=-%3EisAnon%7C-%3EisRegistered%7Cmw%5C.user%5C.isAnon%7Cmw%5C.user%5C.getId%7Cmw%5C.user%5C.getRegistration&files=&excludeFiles=&repos=mediawiki%2Fextensions%2FMachineVision | here ]] ATM Special:SuggestedTags requires a user to be logged in, which suggests that we ought not allow a temp user to use it ... but we should make sure that there's no login required to use the API (so the ISA tool can continue to work). Probably we'll need to replace `isAnon` with `!isNamed`, make sure that an anon/temp user can't reach the `mw.user.getId` code, find where the code checks if a user is logged in, and then test everything to make sure it still works when logged in as a regular user . To enable temporary accounts in your local dev env, particularly **What will be affected**.add this to LocalSettings.php A preliminary investigation (T326759) has found that the following may be affected:``` * MachineVision$wgAutoCreateTempUser['enabled'] = true; ```