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

poudriere.pkg_deps% not MIMEd as plain text, naming is challenging for OS parsing #1135

Open
gessel opened this issue Mar 16, 2024 · 0 comments

Comments

@gessel
Copy link

gessel commented Mar 16, 2024

Prerequisites

  • [ X] Have you checked for an existing issue describing your problem?
  • [ X] Are you running the latest version?
  • [ X] Is your ports tree recent?
  • [ X] Is your FreeBSD Host on a supported release?

Describe the bug

Choosing from the poudriere web interface (nginx), Logs->Dependency Graph, initiates a download of http://server/data/FBSD_14-0-R-default/2024-03-16_01h52m39s/.poudriere.pkg_deps%25

Unlike the log files, which after the suggested minor modification to /usr/local/etc/nginx/mime.types display nicely in the browser, the file is downloaded instead, despite also being plain text and despite it also being convenient to be able to view it in the same browser window.

Further, the file name confuses at least my Linux Mint desktop and makes it ever so slightly less convenient still to see the precious contents therein.

How to reproduce

Steps to reproduce the behavior:

  1. Go to 'Logs->Dependency Graph'
  2. Click on 'Dependency Graph'
  3. See error oopisie

Expected behavior

Show me the log file! In a browser window. optimally a new one, just like logs do.

Screenshots

![image](https://github.com/freebsd/poudriere/assets/3670635/80250f8b-dd26-4eff-ac69-398a6bae8e60)

Environment

  • Host OS [e.g. 12.2 amd64]: 14.0R amd64
  • Jail OS [e.g. 12.0 powerpc]: 14.0R amd64
  • Browser: [e.g. chrome, safari]: brave
  • Poudriere Version [e.g. 3.3.1 or git hash or port version]: 3.4.1_1
  • Ports branch and revision [e.g. 2020Q3 r550754]: main

Additional context

Changing the name of the log to something like "poudriere_pkg_deps.txt" would make it easy.
@gessel gessel added the bug label Mar 16, 2024
@bdrewery bdrewery added the web label Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants