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

[Cockpit 389ds plugin] base64 decode bug #5607

Closed
cold01water opened this issue Jan 17, 2023 · 0 comments
Closed

[Cockpit 389ds plugin] base64 decode bug #5607

cold01water opened this issue Jan 17, 2023 · 0 comments
Assignees
Labels
cockpit Cockpit UI Plugin

Comments

@cold01water
Copy link

Issue Description
There is a bug in the base64 decoding process.

Package Version and Platform:

  • Platform: ALmalinux9
  • Package and version: master branch
  • Browser Edge

Steps to Reproduce

  1. Add attributes containing unicode
  2. When I view the entry, no base64 decoding is done

Expected results
Decoded content should be displayed

Additional context
utils.jsx result should be declared.

export function b64DecodeUnicode (str) {
  // Going backwards: from bytestream, to percent-encoding, to original string.
  try {
      // !!!! put let here !!!!!
      result = decodeURIComponent(atob(str).split('').map(c => {
          return '%' + ('00' + c.charCodeAt(0).toString(16)).slice(-2);
      }).join(''));
      return result;
  } catch(e) {
      console.debug("b64DecodeUnicode failed to decode: ", str);
      return str;
  }
}
@cold01water cold01water added the needs triage The issue will be triaged during scrum label Jan 17, 2023
@mreynolds389 mreynolds389 self-assigned this Jan 20, 2023
mreynolds389 added a commit to mreynolds389/389-ds-base that referenced this issue Jan 23, 2023
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: 389ds#5607
relates: 389ds#5351
relates: 389ds#5611

Reviewed by: spichugi(Thanks!)
mreynolds389 added a commit that referenced this issue Jan 23, 2023
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: #5607
relates: #5351
relates: #5611

Reviewed by: spichugi(Thanks!)
mreynolds389 added a commit that referenced this issue Jan 23, 2023
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: #5607
relates: #5351
relates: #5611

Reviewed by: spichugi(Thanks!)
mreynolds389 added a commit that referenced this issue Jan 23, 2023
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: #5607
relates: #5351
relates: #5611

Reviewed by: spichugi(Thanks!)
mreynolds389 added a commit that referenced this issue Jan 23, 2023
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: #5607
relates: #5351
relates: #5611

Reviewed by: spichugi(Thanks!)
mreynolds389 added a commit that referenced this issue Jan 23, 2023
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: #5607
relates: #5351
relates: #5611

Reviewed by: spichugi(Thanks!)
@mreynolds389 mreynolds389 added cockpit Cockpit UI Plugin and removed needs triage The issue will be triaged during scrum labels Jan 23, 2023
lab-at-nohl pushed a commit to lab-at-nohl/cockpit-389-ds-containerproxy that referenced this issue May 9, 2024
Descriptrion:

5607 - Ldap Editor failed to decode base64 values
5351 - CLI - Cockpit enable check for cockpit package was not portable
       (just removed this check)
5611 - Security page had a lot of issues when trying to change the Server
       Certificate.  Save didn't work, and "Security Enable" modal would
       crash

relates: 389ds/389-ds-base#5607
relates: 389ds/389-ds-base#5351
relates: 389ds/389-ds-base#5611

Reviewed by: spichugi(Thanks!)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cockpit Cockpit UI Plugin
Projects
None yet
Development

No branches or pull requests

2 participants