rustdoc output directory structure leaks home directory structure

[gmjosack]

Since Rust-CI has been continuously failing I've been looking to move my docs to GitHub Pages. This led me to notice that when rustdoc is called with an absolute path as input (as cargo does) the files in doc/src will show your entire home directory layout.

This is problematic not only because of the leak of someone's home directory layout, but because those links will break if I build from a new directory, a new computer, and a different contributor commits those changes. Also if you have 2+ contributors you're going to have massive churn on a repo checking in those files. Least importantly the URLs are ugly when clicking on a src link. :)

As an example on Rust's own API docs, If you click the [src] link on http://doc.rust-lang.org/std/vec/ you end up at http://doc.rust-lang.org/src/collections/home/rustbuild/src/rust-buildbot/slave/nightly-linux/build/src/libcollections/vec.rs.html#11-2901

[lifthrasiir]

I think this is fixed by #19436.

[alexcrichton]

Thanks @lifthrasiir!

[gmjosack]

Thanks @lifthrasiir !

[devsnek]

This appears to be a problem again. I'm seeing stuff like

<a class='srclink' href='../../src/msfs/home/snek/Desktop/projects/msfs-rs/target/debug/build/msfs-7d0246a486c664b1/out/bindings.rs.html#3101' title='goto source code'>[src]</a>

in my output.

[jyn514]

That looks like it's specific to files generated by build scripts, I haven't seen it for most pages.

[GuillaumeGomez]

Isn't it possible only for local builds? On docs.rs and hosted docs, it shouldn't be possible, no?

[jyn514]

Hmm, I thought it was possible on docs.rs, but looking at https://docs.rs/zstd-sys/1.4.17+zstd.1.4.5/zstd_sys/struct.ZSTD_DDict_s.html it looks like the file path is not leaked. I could have sworn I'd seen this on docs.rs before, though.