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

Relative Links to directories cause fatal error #114

Closed
ryan-s opened this issue Sep 9, 2021 · 4 comments
Closed

Relative Links to directories cause fatal error #114

ryan-s opened this issue Sep 9, 2021 · 4 comments
Labels

Comments

@ryan-s
Copy link

ryan-s commented Sep 9, 2021

Having a line like:
[`./integration`](integration/):` Integration tests
that is a relative link to a directory in a repository causes the following fatal error:

2021-09-09 13:23:33.173 TRACE found a relative link: full=integration filename=integration hash=
2021-09-09 13:23:33.173 FATAL unable to resolve relative links
                              └─ resolve link: "integration"
                                 └─ read file: integration
                                    └─ read integration: is a directory

Expected: Relative links that are directories are ignored
Actual: Relative links that are directories cause a fatal error, preventing file from syncing to confluence.

@ryan-s
Copy link
Author

ryan-s commented Sep 9, 2021

Looking at this the error seems to be coming from Karma. This made me unsure if this was expected behavior or behavior that was inherited from the dependency.

@kovetskiy
Copy link
Owner

kovetskiy commented Sep 9, 2021 via email

@ryan-s
Copy link
Author

ryan-s commented Sep 9, 2021

I can think of 2 potential behaviors:

  1. I think the base case would be just to ignore the link and only upload the text to confluence
  2. Alternatively, and this would probably be more of a feature-request, you could give the user the ability to pass a hosted repository url as an argument and it would generate a link to the directory url inside of the hosted repository.

@kovetskiy kovetskiy assigned kovetskiy and unassigned kovetskiy Sep 11, 2021
@kovetskiy kovetskiy added the bug label Sep 11, 2021
@kovetskiy
Copy link
Owner

Should be fixed in version 6.3, let me know if it doesn't work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants