Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Basic SQS integration #38

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Basic SQS integration #38

wants to merge 8 commits into from

Conversation

selul
Copy link
Contributor

@selul selul commented Jul 16, 2018

I've added a basic SQS integration, following the Rabbitmq example.

It requires defining an $aws_credentials array having this format:

global $aws_credentials;
$aws_credentials = array(
	'region'      => 'us-east-1',
	'version'     => 'latest',
	'credentials' => array(
		'key'    => '', //AWS key.
		'secret' => ' ' //AWS secret.
	)
);

The queue name is hardcoded to wordpress.
The PR implements the system tests also, but it will require to define in config.php file the $aws_credentials array.

selul and others added 4 commits July 14, 2018 21:38
Implements SQS Worker, Client and Connection class.
Adds global variable $aws_credentials as well as sqs for WP_MINIONS_BACKEND.
Remove testing queue name, adds wordpress, following the rabbitmq example.
@jeffpaul jeffpaul added this to the 4.2.0 milestone Sep 24, 2019
@jeffpaul jeffpaul requested a review from tlovett1 September 24, 2019 13:49
@jeffpaul
Copy link
Member

@selul can you look into resolving the merge conflicts here?

@selul
Copy link
Contributor Author

selul commented Sep 25, 2019

@jeffpaul I've just done it, thanks for heads-up.

@jeffpaul jeffpaul requested a review from dinhtungdu March 21, 2020 22:36
@jeffpaul jeffpaul changed the base branch from master to develop May 25, 2021 19:25
@dkotter dkotter removed the request for review from dinhtungdu August 23, 2022 22:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants