Skip to content

Replaced httpunit by htmlunit#190

Merged
arun-gupta merged 1 commit intojavaee-samples:masterfrom
arjantijms:master
Jan 11, 2014
Merged

Replaced httpunit by htmlunit#190
arun-gupta merged 1 commit intojavaee-samples:masterfrom
arjantijms:master

Conversation

@arjantijms
Copy link
Contributor

Added convenience method for getting raw content from server and disabled throwing exception upon 403.

content from server and disabled throwing exception upon 403.
arun-gupta added a commit that referenced this pull request Jan 11, 2014
Replaced httpunit by htmlunit
@arun-gupta arun-gupta merged commit 0af95ee into javaee-samples:master Jan 11, 2014
@arjantijms
Copy link
Contributor Author

Hi,

All the tests are now failing with this;

java.lang.Exception: {"JBAS014771: Services with missing/unavailable dependencies" => ["jboss.undertow.deployment.default-server.default-host./b86b0b08-78c7-4970-9ac3-3fb77cead7a3.UndertowDeploymentInfoService is missing [jboss.security.security-domain.jaspitest]"]}

See e.g. https://arungupta.ci.cloudbees.com/job/Java%20EE%207%20Samples%20on%20WildFly-cb/lastCompletedBuild/testReport/org.javaee7.jaspic.ejbpropagation/ProtectedEJBPropagationTest/org_javaee7_jaspic_ejbpropagation_ProtectedEJBPropagationTest/

Most likely the dummy security domain in standalone.xml has to be added again. It's this configuration:

 <security-domain name="jaspitest" cache-type="default">
                <authentication-jaspi>
                    <login-module-stack name="dummy">
                        <login-module code="Dummy" flag="optional"/>
                    </login-module-stack>
                    <auth-module code="Dummy"/>
                </authentication-jaspi>
            </security-domain>

Incidentally, this is why this kind of config should not be necessary. It's too easy to forget and especially new users will be baffled by it. As it's just a dummy config that essentially does nothing the user should also not be asked to add it. But alas... adding it for now should hopefully improve the test results again.

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.

2 participants