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.

Sign up for GitHub

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

Jump to bottom

Namespace of GlassFish deployment descriptors needs changing #23014

Open
smillidge opened this issue May 18, 2020 · 5 comments
Open

Namespace of GlassFish deployment descriptors needs changing #23014

smillidge opened this issue May 18, 2020 · 5 comments
Projects
6.0.0 Release

Comments

@smillidge
Copy link
Contributor

Deployment descriptor DTDs and schemas should be updated to reference the new website and placed here https://eclipse-ee4j.github.io/glassfish/dtds/glassfish
@smillidge smillidge added this to the 6.0.0 milestone May 18, 2020
@smillidge smillidge added this to To do in 6.0.0 Release May 18, 2020
@gerdogdu
Copy link
Contributor

gerdogdu commented May 18, 2020 •
edited

Which version of DTDs must be change? do we need to change these for example

DOCTYPE ejb-jar PUBLIC '-//Sun Microsystems, Inc.//DTD Enterprise JavaBeans 2.0//EN' 'http://java.sun.com/dtd/ejb-jar_2_0.dtd'

Is there any common location for DTDs?

@nitind
Copy link

nitind commented Jul 23, 2020 •
edited

The redirection of http://glassfish.org/dtds/* also needs to be updated. Right now it (everything from glassfish.org?) just goes to the home page, breaking validation of deployment descriptors.

@arjantijms arjantijms modified the milestones: 6.0.0, 6.1.0 Jan 1, 2021
@arjantijms arjantijms modified the milestones: 6.1.0, 6.1.1 May 19, 2021
@arjantijms arjantijms removed this from the 6.2 milestone Jun 13, 2021
@hs536
Copy link
Contributor

hs536 commented Jun 14, 2021

I think the DTDs that needs to be updated is the deployment descriptors that GlassFish defines itself.
They are:

Defined in:

  • appserver/deployment/dol/src/main/java/com/sun/enterprise/deployment/xml/DTDRegistry.java

@github-actions
Copy link

This issue has been marked as inactive and old and will be closed in 7 days if there is no further activity. If you want the issue to remain open please add a comment

@github-actions github-actions bot added the Stale label Jun 15, 2022
@hs536 hs536 mentioned this issue Oct 24, 2022
Closed
@dmatej dmatej removed the Stale label Oct 24, 2022
@dmatej dmatej reopened this Oct 24, 2022
@dmatej dmatej added this to the 7.0.0 milestone Oct 24, 2022
@arjantijms arjantijms removed this from the 7.0.0 milestone Dec 14, 2022
OndroMih added a commit to OndroMih/glassfish that referenced this issue May 27, 2023
@OndroMih
Add dtds reported in eclipse-ee4j#23014 and eclipse-ee4j#23665 to the…
99f611a 
… website

eclipse-ee4j#23665
eclipse-ee4j#23014
Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>
hs536 added a commit that referenced this issue May 28, 2023
@hs536
Merge pull request #24436 from OndroMih/ondromih-add-dtds-to-website
b68c29e 
Add dtds reported in #23014 and #23665 to the website
@github-actions GitHub Actions
Copy link

This issue has been marked as inactive and old and will be closed in 7 days if there is no further activity. If you want the issue to remain open please add a comment

@github-actions github-actions bot added the Stale label Dec 15, 2023
@dmatej dmatej removed the Stale label Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
No open projects
6.0.0 Release
  
To do
Milestone
No milestone
Development

No branches or pull requests
6 participants
@dmatej @smillidge @gerdogdu @nitind @arjantijms @hs536