This is a report on what I plan to work on this week in open source. I signed up for work on two issues.
Hurl: --cookie-jar FILE doesn't work if intermediate directories don't exist
The first is an issue in Hurl, an HTTP request/response client and assertion tool.
--cookie-jar FILE doesn't work if intermediate directories don't exist
#3637
When exporting cookies from a run to SOME_FILE export is failing if SOME_FILE has non-existing directory:
$ echo 'HEAD https://hurl.dev' | hurl --cookie-jar /tmp/a/b/c/out.txt
error: Issue writing to /tmp/a/b/c/out.txt: Os { code: 2, kind: NotFound, message: "No such file or directory" }
--cookie-jar should works like --curll which create intermediate directories:
$ echo 'HEAD https://hurl.dev' | hurl --curl /tmp/a/b/c/out.txt
Note: we could try while fixing this bug to mutualize the code that create a file and it's indermediate directories:
// We ensure that parent folder is created.
if let Some(parent) = filename.parent() {
match std::fs::create_dir_all(parent) {
Ok(_) => {}
Err(err) => {
return Err(ReportError::from_error(
err,
filename,
"Issue writing TAP report",
))
}
}
}
let mut file = match File::create(filename) {
Ok(f) => f,
Err(e) => {
return Err(ReportError::from_error(
e,
filename,
"Issue writing TAP report",
))
}
};
This issue concerns the command --cookie-jar <FILE>. If the file path specified contains intermediate directories that doesn't exist, it throws an error. We want it to automatically create the intermediate directories.
Hurl is a tool that I use (and still use) in my own projects. I want to continue doing meaningful work and I thought this would be a good way to get back into the flow of things since I haven't done much open source work since December. This will also be my first time working with Rust, which I'm looking forward to.
Starchart: Add ability to create and manage MX records
The second issue I signed up for is an issue in Starchart, our in-house tool for creating and managing custom subdomains.
Add ability to create and manage MX records
#787
Currently you can create the following types of DNS records:
- A
- AAAA
- CNAME
- TXT
A request has come in to support also including MX records for defining names for email hosts.
We want to add support for MX records, a type of DNS record for mail servers. I'm not too familiar with how the app manages DNS records so I'll have to do some research. This issue seems rather large in scope, so while I will be working on the implementation, I don't expect to finish it within the week. My professor also suggested I split it up into smaller issues - writing tests and doing the actual implementation - and let somebody else do the other half.
I signed up for this because we've been updating dependencies in this project for a couple weeks now to bring it up to date, but haven't had a chance to dive into the actual code, so I thought it'd be nice to finally do just that.
I did do some other minor stuff like cleaning up .prettierignore, adding an .nvmrc file, adding docker compose down to the contributing docs, and organizing the project structure a little.
Move docs into /docs
#810
I think it'd be nice to clean the project root directory up to make it a little easier to navigate. This moves the documentation and images into /docs so they're not mixed in with config files and such.
Move mysql and redis bind mounts to /docker/volumes
#813
This aims to make the project root directory a bit more organized by moving all bind mounts under a single directory.
I'll do a progress report on Wednesday with how the issues are going.
Top comments (0)