Current Drupal 8 version is unusable right now.
It looks like the module is unmaintained for years already.

I have forked and deeply re-factored it here: https://github.com/milkovsky/aws_sqs/tree/feature/8.x-improvements.
Please concieder merging the PR: https://github.com/skwashd/aws_sqs/pull/2.

Comments

a.milkovsky created an issue. See original summary.

malcolm_p’s picture

malcolm_p commented

I would also like to see a D8 release of this module. If this project is no longer being supported it'd be great to get some new maintainers. I'm willing to help as I plan to use AWS SQS extensively in D8.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

Happy to give you maintainership! Would you be ok if I assign the project to you?

malcolm_p’s picture

malcolm_p commented

Thanks, that'd be great!

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Active » Fixed

Malcolm, you have maintainership now!

a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented
Status: Fixed » Needs work

Thanks for adding a new maintainer.
The initial issue is about re-factoring, so we can not close it.

malcolm_p’s picture

malcolm_p commented

Thanks Nick! Alex, hopefully I'm going to be working on this starting next week.

asauterChicago’s picture

asauterChicago commented

Hello, I'm not sure what the process on this is, but I would also like maintainership. I would like to test and merge this pull request. And I would like to add some new features that are available in the latest SDK and fix a bunch of code to be in line with the latest Drupal standards.

bunty badgujar’s picture

bunty badgujar commented

Hello @Nick_vh, Any progress in this module? I am working on the same feature as this module. I would like to contribute my time. Can you give me maintainership. So that i can update module code for drupal 8 and 9.

bunty badgujar’s picture

bunty badgujar commented
Assigned: Unassigned » bunty badgujar
nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

Maintainership transferred, let's hope to see some life in this module again :-)

bunty badgujar’s picture

bunty badgujar commented
Status: Needs work » Fixed
bunty badgujar’s picture

bunty badgujar commented
Status: Fixed » Closed (fixed)
a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented

As written in #6
Thanks for adding a new maintainer.
The initial issue is about re-factoring, so we can not close it!
Please re-open the issue. The PR still needs review.

bunty badgujar’s picture

bunty badgujar commented

Hello @Alex Milkovskyi,
In issue description it is written that the module is unstable for drupal 8. But as you can see, module now have stable release and working fine on default queue and with service. So, can you please provide patch of the re-factored code?

Also feel free to reopen this issue.

a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented

Also feel free to reopen this issue.

I can not reopen this issue, because it is closed. I do not have permissions.

In issue description it is written that the module is unstable for drupal 8.

Yes I know, I created this issue :D
As you can see there is a pull request in the description. It is on github, because module development was on github before.

I was expecting some feedback on the proposed code changes.
But too late now, you made too many changes already.

bunty badgujar’s picture

bunty badgujar commented

@Alex Milkovskyi, As most of the changes you proposed is already added in current release. But, i can see following interesting code in your PR which can include in this module :-

  1. Better factory method to avoid multiple queue initialization. Can use protected $initializedQueues;
  2. Use aws Credentials class for credential. Also we can use "CredentialProvider" to support IAM roles as well.
  3. Add more aws sqs region.

Let me know if you want to track these changes in separate issue or this issue? Also credit for this issue has been given to all contributors. :)

a.milkovsky’s picture

a.milkovsky
he/him
Location UTC +2
commented

Add more aws sqs region.

Yes, they have added quite a few. "'eu-central-1' => 'EU (Frankfurt)'" is an important one for the GDPR topic in Europe.

Let me know if you want to track these changes in separate issue or this issue?

It is up to you. If you would like to create a new issue, just reference it here in a comment.

FYI I will not contribute a new patch any soon, because currently I do not have any related project.

Thanks for the review!

bunty badgujar’s picture

bunty badgujar commented

As discussed in #17