Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clarify adding external connections instructions in README, fix #837 #838

Merged
merged 3 commits into from
Feb 24, 2022
Merged

clarify adding external connections instructions in README, fix #837 #838

merged 3 commits into from
Feb 24, 2022

Conversation

skorasaurus
Copy link
Contributor

@skorasaurus skorasaurus commented Jan 28, 2022

Description of the Change

Clarified the instructions for setting up External Connections. Existing instructions assumes use of the applications-password plugin whose functionality has been merged into WordPress core as of 5.6.
The current Add New External Connection page is also different than what is described in the readme and instructions have been updated for that (more information in #837

Closes #837

Alternate Designs

Possible Drawbacks

Maybe you want users to use the application-password plugin? Additional references to using the application-password are still in README and readme.

Verification Process

What process did you follow to verify that your change has the desired effects?

  • Manually followed my new instructions on 5.8.3 websites.

Checklist:

  • [ x] I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • [x ] My change requires a change to the documentation.
  • [x ] I have updated the documentation accordingly.
  • [ na] I have added tests to cover my change.
  • All new and existing tests passed.

Changelog Entry

Changed - documentation updates.

Credits

Props @skorasaurus, @jeffpaul.

@skorasaurus skorasaurus changed the title clarify adding external connections instructions, fix #837 clarify adding external connections instructions in README, fix #837 Feb 2, 2022
Copy link
Member

@jeffpaul jeffpaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@skorasaurus thanks for getting this started, much appreciated! Things should be even more simple than what you've updated to here, so if you're able to update your PR to the non-manual connection setup to use the Remove Connection Wizard steps that would be great... thanks again!

README.md Outdated
1. On remotesite.com, ensure that [Application Passwords](https://wordpress.org/plugins/application-passwords/) is installed. (_Note: Using this plugin instead of a normal WordPress users password helps limit the use of your primary password and will allow you to revoke access to Distributor in the future if needed._) Then navigate to the user profile that will be used to create the External Connection on mainsite.com and then to the `Application Passwords` section of the user profile (not the `Account Management` section). Add a label for the New Application Password Name (e.g., `mainsite.com`) and click `Add New`. Now copy the password provided into mainsite.com's External Connections `Password` field.
1. On mainsite.com, add the `External Connection URL` (e.g., http://remotesite.com/wp-json). You should see a green circle and "_Connection established._".
1. Enter a label for the connection (e.g., `remotesite.com`).
1. Select `Username / Password` for the `Authentication Method`, open the `manually set up connection` link, and input a username from remotesite.com.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This and the next couple steps can generally be skipped as the "Remote Connection Wizard" helps skip many of these steps. Should essentially be (1) add label for the External Connection, (2) enter domain for the external site, (3) click Authorize Connection, (4) process acceptance from there to get to the final step of saving the new External Connection.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I had a misunderstanding of how Application Passwords were being used and did not think that an Application Password.

As I understand now, an application password is now automatically generated and saved when going through the "Remote Connection Wizard" process although the user first enters their user name and their password (not an application password) when making the authorization (if the user is not already logged into that other website). At the time of making the PR, I didn't know that (application password creation) was happening behind the scene.

The message in the 'Edit External Connection' page -
"Important: We strongly recommend using the Application Passwords plugin on the site you are connecting to in order to create a unique password for this connection. This helps limit the use of your primary password and will allow you to revoke access in the future if needed." -

led me to believe that my credentials were being stored instead (of a unique application password) ; as indicated in the screenshot below

Selection_148

I've updated the PR using the "Remote connection Wizard" directions.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@skorasaurus good callout on the text on the Edit External Connection page, we could/should probably remove that entire line at this point as that adds unnecessary plugins to someone's site as WordPress core now handles all that functionality. If you'd like, I'd say go ahead and strip that out as part of this PR as well if you'd like?

@skorasaurus skorasaurus requested a review from jeffpaul February 10, 2022 00:40
README.md Outdated Show resolved Hide resolved
@jeffpaul jeffpaul self-requested a review February 24, 2022 18:23
Copy link
Member

@jeffpaul jeffpaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @skorasaurus, much appreciated!

@jeffpaul jeffpaul merged commit 3ab613c into 10up:develop Feb 24, 2022
@dkotter dkotter modified the milestones: 1.7.0, 1.6.9 Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

README install assumes use of WordPress <5.6 and use of application-passwords plugin
3 participants