Annaxs commented on issue #23836: URL: https://github.com/apache/pulsar/issues/23836#issuecomment-4377146001
Hi, I'd like to work on this issue. Here's my analysis:
▎ Root cause — In src/components/downloads.tsx, the
CppReleasesDownloadTable function builds a url (directory path) and a tarPath
(actual file), but
mistakenly uses url for the link field instead of tarPath:
▎ // Current (broken) — link points to directory
▎ link: url, //
https://archive.apache.org/dist/pulsar/pulsar-client-cpp-4.1.0/
▎ linkText: `apache-pulsar-cpp-${version}`, // also incorrect filename
▎ // Fix — link points to the actual source tarball
▎ link: tarPath, //
https://archive.apache.org/dist/pulsar/pulsar-client-cpp-4.1.0/apache-pulsar-client-cpp-4.1.0.tar.gz
▎ linkText: `apache-pulsar-client-cpp-${version}.tar.gz`,
▎ The asc and sha512 fields already correctly use tarPath — only link and
linkText need fixing. I verified the source tarball exists in the archive for
both 3.x and 4.x versions.
▎ I also noticed GoReleasesDownloadTable has the identical bug (same
pattern, verified tarball exists in archive). I plan to fix both in the same
PR.
▎ Could this be assigned to me? Thanks!
@lhotari can i get this assigned?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
