Project

General

Profile

Actions
Copy link

Bug #34681

closed

Updates oauth Gem installed by puppet-agent-oauth under Debian to a newer version (0.5.1 produces lots of warnings about URI.escape)

Added by Andreas Ntaflos about 3 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Debian/Ubuntu
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman-packaging/pull/7745, https://github.com/theforeman/foreman-packaging/pull/9381
Fixed in Releases:
Foreman - 3.1.3, Foreman - 3.2.1, Foreman - 3.3.0
Found in Releases:
Foreman - 3.1.2
Red Hat JIRA:

Description

Under Debian/Ubuntu the .deb package puppet-agent-oauth installs the oauth Gem in Puppet's Ruby environment by means of a post-install hook into /opt/puppetlabs/puppet/lib/ruby/gems/2.7.0/gems/oauth-0.5.1.

This version of oauth is old and produces annoying warnings (about 100 lines) under Puppet 7 during every Puppet agent run on any node that runs foreman-proxy:

/opt/puppetlabs/puppet/lib/ruby/gems/2.7.0/gems/oauth-0.5.1/lib/oauth/helper.rb:12: warning: URI.escape is obsolete
/opt/puppetlabs/puppet/lib/ruby/gems/2.7.0/gems/oauth-0.5.1/lib/oauth/helper.rb:12: warning: URI.escape is obsolete
/opt/puppetlabs/puppet/lib/ruby/gems/2.7.0/gems/oauth-0.5.1/lib/oauth/helper.rb:12: warning: URI.escape is obsolete
/opt/puppetlabs/puppet/lib/ruby/gems/2.7.0/gems/oauth-0.5.1/lib/oauth/helper.rb:12: warning: URI.escape is obsolete
...

I see that under RedHat the oauth Gem is installed in a newer version (0.5.5) that should not produce any warnings related to URI.escape so it seems to me that there is no reason to keep the oauth Gem at 0.5.1 under Debian.

I can provide a PR that does the work of updating oauth to 0.5.5 but I am not 100% on the Foreman project's packaging workflows.

Actions
Copy link
 #2

Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-packaging/pull/7745 added
Actions
Copy link
 #3

Updated by The Foreman Bot about 3 years ago

  • Fixed in Releases 3.3.0 added
Actions
Copy link
 #4

Updated by Ewoud Kohl van Wijngaarden about 3 years ago

  • Category set to Debian/Ubuntu
  • Triaged changed from No to Yes
  • Fixed in Releases 3.1.3, 3.2.1 added
Actions
Copy link
 #5

Updated by Anonymous about 3 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link
 #6

Updated by Amit Upadhye almost 3 years ago

  • Subject changed from Should update oauth Gem installed by puppet-agent-oauth under Debian to a newer version (0.5.1 produces lots of warnings about URI.escape) to Updates oauth Gem installed by puppet-agent-oauth under Debian to a newer version (0.5.1 produces lots of warnings about URI.escape)
Actions
Copy link
 #7

Updated by The Foreman Bot almost 2 years ago

  • Pull request https://github.com/theforeman/foreman-packaging/pull/9381 added
Actions
Copy link

Also available in: Atom PDF