Skip to content
This repository has been archived by the owner on Mar 29, 2023. It is now read-only.

this.config not set #33

Closed
spearce opened this issue Dec 2, 2016 · 5 comments
Closed

this.config not set #33

spearce opened this issue Dec 2, 2016 · 5 comments

Comments

@spearce
Copy link
Contributor

spearce commented Dec 2, 2016

Originally reported on Google Code with ID 33

What steps will reproduce the problem?
1. deploy application
2. browse first page


What is the expected output?
working index page

What do you see instead?
java.lang.NullPointerException: cfg


What version of the product are you using?
commit 076b1bf9bb40bd653ff2642903c7efdd8bb36d32

 On what operating system?
x86_64-freebsd

Please provide any additional information below.
com.google.gitiles.GitilesFilter.setDefaultFields fails to set this.config if it is
null.


Reported by None on 2013-10-24 09:47:43

@spearce
Copy link
Contributor Author

spearce commented Dec 2, 2016

What steps will reproduce the problem?
1. deploy application
2. browse first page


What is the expected output?
working index page

What do you see instead?
java.lang.NullPointerException: cfg


What version of the product are you using?
commit 076b1bf9bb40bd653ff2642903c7efdd8bb36d32

 On what operating system?
x86_64-freebsd

Please provide any additional information below.
com.google.gitiles.GitilesFilter.setDefaultFields fails to set this.config if it is
null.


Reported by None on 2013-10-24 09:47:43

@spearce
Copy link
Contributor Author

spearce commented Dec 2, 2016

The attached patch fixes the issue.

Reported by None on 2013-10-24 09:48:05


* *Attachment: [0001-Set-this.config-at-initialization.patch](https://storage.googleapis.com/google-code-attachments/gitiles/issue-33/comment-1/0001-Set-this.config-at-initialization.patch)*

@spearce
Copy link
Contributor Author

spearce commented Dec 2, 2016

This is with Jetty 8.1.11.

Reported by None on 2013-10-24 09:50:41

@spearce
Copy link
Contributor Author

spearce commented Dec 2, 2016

This appears to have been fixed in b5069825016ddef40875f40126fd71193bbb7f43

Reported by None on 2014-11-12 06:06:22

@spearce
Copy link
Contributor Author

spearce commented Dec 2, 2016

Reported by None on 2015-06-23 22:39:33

@spearce spearce closed this as completed Dec 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant