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

replace http with https for dtd file paths #2462

Closed
wants to merge 3 commits into from
Closed

Conversation

vsp-gleich
Copy link
Contributor

@vsp-gleich vsp-gleich requested a review from kainagel March 22, 2023 18:45
@tschlenther
Copy link
Contributor

@vsp-gleich do we still need this PR ord can you close it?

@kt86
Copy link
Contributor

kt86 commented Aug 6, 2024

I think that this makes totally sense. I also stumbled over this issue a few weeks ago. During that I tried to change it at least for some freight related stuff, but cancelled it, because currently we cannot automatically update the DTD/XSD files on matsim.org/files/dtd ..
(and in these files all the things must be updated as well).

So from my point of view: I make sense to update, but let's wait until the automatic update of the files works again.
(And maybe restart with a new PR instead of trying to fix this one)

@vsp-gleich
Copy link
Contributor Author

We can close this PR, it did not work out a year ago and now there are too many conflicts.

@vsp-gleich vsp-gleich closed this Aug 6, 2024
@vsp-gleich vsp-gleich deleted the httpsdtdpaths branch August 6, 2024 12:02
@sebhoerl sebhoerl mentioned this pull request Aug 19, 2024
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