Bail from try_xdg if browser name is empty

[krsh732]

Hey, not sure if I'm horribly off the mark. Had an infinite loop in WSL with the following on main branch:

fn main() -> Result<(), std::io::Error> {
    println!("HI");
    webbrowser::open("https://youtube.com")?;
    println!("BYE");
    Ok(())
}

There was apparently no default for browser in xdg settings:

This causes an infinite loop because open_using_xdg_config ends up opening a directory and rust-lang/rust#64144?

Fixed it by simply bailing if there is no name.

[amodm]

Thanks for your first contribution here, @krsh732. LGTM.

[amodm]

@krsh732, I'm unable to test this manually due to some issues with my Windows VM. I can release this only after confirmation that post this change, it did work for you, as in the non-xdg codepath kicked in, and opened the default windows browser.

Can you please confirm?

[krsh732]

@amodm, Yeah can confirm that the browser opens into the right change after this change:

[amodm]

Awesome, thanks. This is now out as v0.8.7