OAuth 2.0 support? (D10+)

As stated "changes immediately" are needed and is crtical. We have now same issue as well.
Is there any news about that?
What i see is there XML used: https://git.drupalcode.org/project/commerce_ups/-/blob/8.x-3.x/src/UPSRa...

As stated composer: https://git.drupalcode.org/project/commerce_ups/-/blame/8.x-3.x/composer...
This library don't support oAuth and is abandom: https://github.com/gabrielbull/php-ups-api/issues/370

There is new library needed to be incorporate:
https://github.com/abantecart/ups-php

It's now only one to be incorporate to existing package and set as v4 version to difference for change of API provider

I ask about difference: https://github.com/abantecart/ups-php/issues/7
We should only replace API as issue is critical and required to be solved in fastest manner to have 1 from 2 available shipping integration provider

Maintainer suggest only to swap library and maintaining will be provided n future : https://github.com/abantecart/ups-php/issues/7#issuecomment-1832083838

Given that D9 is no longer supported..

@DamienMcKenna do you need help? Are you into this issue? It block whole Commerce deployment as now is don't have one production grade shipping method integration

Sorry, but I'm not in a position to work on this.

Ryan, this is a total show stopper issue to our project. UPS will not issue an old style API key and we have to have UPS shipping on our rewards portal. Please can you let us know if we can help with any developmemp?

https://www.drupal.org/u/jsacksick → can you check it? This can require to replace whole API and fixes for old one as well as XML method will require to be removed

It's definitely not a drop in replacement. I'm not sure how different the two libraries are but I can say their folder structure and class names are not the same.

Hi, any update on this? till can we expect a release with the OAuth functionality, thanks

Thank you tBKoT!

Might anyone have some time to test this out?

I should be able to test this last this week.

I've reviewed the patch; it's functioning well overall. However, there are some key points to address:

1. To avoid errors, implement the solution found here: 🐛 Error: Call to a member function getLength() Needs review
2. If utilizing any default packages, refer to this important note: ✨ Default UPS packages types need updating Active
3. The issue with Negotiated Rates not working has been resolved; I've already committed the fix to the merge request.

Testing Standard Rates vs. Negotiated Rates:

Hi,
I´m still having problems with negotiated rates.
After applying latest patch 7 to the (also latest) dev-Version.

Actually, rates get reduced from e.g. 47.07 (= standard rate) to 46.60 (= negotiated).
But negotiated should go to something like 18.81

I´m sure I do use the correct credentials (especially account number).

Do I have to use any other patches besides this 7.patch to get this solved?

Thanks!

@tBKoT Is it okay to commit this change to UPSSdk.php so we can enable translation of the service names? ( ✨ Allow the UPS service names to be translated Active )?

    public function getServiceName(string $service): string {
        return isset($this->serviceNames[$service]) ? t($this->serviceNames[$service]) : '';
    }

Merged the MR, thanks everyone!

Automatically closed - issue fixed for 2 weeks with no activity.