Skip to content

Commit

Permalink
fix: port 80 ignored in ingestor endpoint (#1004)
Browse files Browse the repository at this point in the history
current state -
    if ingest node starts at port 80 i.e.
    if env vars P_ADDR or P_INGESTOR_ENDPOINT are set with port 80
    server ignores the port and panics at start with error

    fix -
    if port is ignored, use default port to 80
  • Loading branch information
nikhilsinhaparseable authored Nov 27, 2024
1 parent 7b8398d commit 7be5d7e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion server/src/storage/staging.rs
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ pub fn get_ingestor_info() -> anyhow::Result<IngestorMetadata> {
// all the files should be in the staging directory root
let entries = std::fs::read_dir(path)?;
let url = get_url();
let port = url.port().expect("here port should be defined").to_string();
let port = url.port().unwrap_or(80).to_string();
let url = url.to_string();

for entry in entries {
Expand Down

0 comments on commit 7be5d7e

Please sign in to comment.