Skip to content

Remove all the formatting characters from the id to prevent invalid url generation#833

Merged
essobedo merged 1 commit intomainfrom
url-generation-error
Apr 27, 2022
Merged

Remove all the formatting characters from the id to prevent invalid url generation#833
essobedo merged 1 commit intomainfrom
url-generation-error

Conversation

@essobedo
Copy link
Contributor

@essobedo essobedo commented Apr 26, 2022

Motivation

If the text provided to the boldLink function contains special formatting characters of asciidoc, the generation of the URL fails, consequently the website cannot be built due to an error of the next type:

[check:links ] Found invalid urls in components/next/mongodb-component.html:
[check:links ] Fragment #_endpoint_header%3C/em%3Eid at components/next/mongodb-component.html does not exist!

In the previous example the provided text is _id which contains an underscore that is one of the formatting character of asciidoc

Modification

  • Update the boldLink function to remove all the special characters from the provided text used to generate the link

@essobedo
Copy link
Contributor Author

@essobedo essobedo force-pushed the url-generation-error branch from abe9d3f to 3a3fd84 Compare April 27, 2022 10:31
@github-actions
Copy link
Contributor

🚀 Preview is available at

@essobedo essobedo merged commit 04f9e4e into main Apr 27, 2022
@essobedo essobedo deleted the url-generation-error branch April 27, 2022 12:27
essobedo added a commit to apache/camel that referenced this pull request Apr 27, 2022
## Motivation

```
public static final String MONGO_ID = "_id"; // default id field
```
Causes this problem while generating the website
```
[check:links ] Found invalid urls in components/next/mongodb-component.html:
[check:links ] Fragment #_endpoint_header%3C/em%3Eid at components/next/mongodb-component.html does not exist!
```

## Modifications

* Re-declare `_id` as an header of Camel MongoDB as the actual fix is in apache/camel-website#833
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants