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

Utils.getCarbonConfigHome() throws an NPE in non-Carbon environments #1309

Open
gokul opened this issue Feb 28, 2017 · 1 comment
Open

Utils.getCarbonConfigHome() throws an NPE in non-Carbon environments #1309

gokul opened this issue Feb 28, 2017 · 1 comment
Labels
C5

Comments

@gokul
Copy link
Member

gokul commented Feb 28, 2017 •
edited
Loading

In a piece of code which can be common to Carbon and non-Carbon environments, the usage of Utils.getCarbonConfigHome() throws an NPE in non-Carbon environments since the System.getenv(Constants.CARBON_HOME_ENV) call fails.

Ideally, in such cases, an exception should be thrown so that the calling party can proceed accordingly.
gokul added a commit to gokul/carbon-kernel that referenced this issue Feb 28, 2017
@gokul
Fixing wso2#1309
f25b324
@gokul
Copy link
Member Author

gokul commented Feb 28, 2017

Added fix in #1310

@gokul gokul mentioned this issue Feb 28, 2017
Closed
@Niranjan-K Niranjan-K added this to the 5.2.0-m4 milestone Feb 28, 2017
@Kishanthan Kishanthan modified the milestones: 5.2.0-m4, 5.2.0-m5 Apr 7, 2017
@cnapagoda cnapagoda removed this from the 5.2.0-m5 milestone Aug 10, 2017
@jsdjayanga jsdjayanga added the C5 label Jul 4, 2019
shashimalcse pushed a commit to shashimalcse/carbon-kernel that referenced this issue Jul 17, 2023
@sachiniWettasinghe
Merge pull request wso2#1309 from sachiniWettasinghe/sw-support-4.6.1…
d62f05f 
….x-full

[UMT - 5.11.0] Use equal operator instead of LIKE to filter users based on an attribute
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C5
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Kishanthan @gokul @jsdjayanga @cnapagoda @Niranjan-K