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

🐛 New Bug: Null pointer Exception is thrown when path() method is not given in the APIRequest #110

Closed
mfaisalkhatri opened this issue Jul 22, 2022 · 0 comments · Fixed by #113
Closed

🐛 New Bug: Null pointer Exception is thrown when path() method is not given in the APIRequest #110

mfaisalkhatri opened this issue Jul 22, 2022 · 0 comments · Fixed by #113
Assignees
@WasiqB
Labels
🛠 goal: fix Bug fix 🏷 status: label work required Needs proper labelling before it can be worked on 🏁 status: ready for dev Ready for work
Milestone
v-0.8.0

Comments

@mfaisalkhatri
Copy link
Contributor

❓ What is the bug?

NPE is thrown when path() method is not given in APIRequest

Version

0.1.0 (Default)

🖥️ What framework domain type is this bug for?

API

📃 Steps to reproduce the bug

While creating an API request, do not provide the path() method.
Currently Base URI from my API is https://restful-booker.herokuapp.com and path is booking which is already updated in the boyka-config.json file. Hence path() method is not required to be given while creating APIRequest.

✅ Expected Behavior

While creating APIRequest path() method should be optional.

❌ Actual Behavior

Following error is thrown when path() method is not given

java.lang.NullPointerException: Cannot invoke "String.contains(java.lang.CharSequence)" because "value" is null
@mfaisalkhatri mfaisalkhatri added 🛠 goal: fix Bug fix 🏁 status: ready for dev Ready for work 🏷 status: label work required Needs proper labelling before it can be worked on labels Jul 22, 2022
@mfaisalkhatri mfaisalkhatri added this to the v-0.8.0 milestone Jul 22, 2022
@WasiqB WasiqB moved this from 💡 Backlog to 🧑‍🏭 Ready for development in 🥊 Boyka-Framework Roadmap Jul 23, 2022
WasiqB added a commit that referenced this issue Jul 24, 2022
@WasiqB
fix(java): 🐛 fixed null pointer error when path method is skipped
46ce9ce 
Closes #110
@WasiqB WasiqB mentioned this issue Jul 24, 2022
Merged
20 tasks
@WasiqB WasiqB closed this as completed in 72fcd99 Jul 24, 2022
Repository owner moved this from 🧑‍🏭 Ready for development to ✅ Done in 🥊 Boyka-Framework Roadmap Jul 24, 2022
@WasiqB WasiqB self-assigned this Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WasiqB WasiqB

Labels
🛠 goal: fix Bug fix 🏷 status: label work required Needs proper labelling before it can be worked on 🏁 status: ready for dev Ready for work
Projects
🥊 Boyka-Framework Roadmap
Status: ✅ Done
Milestone
v-0.8.0
Development

Successfully merging a pull request may close this issue.

fix(java): 🐛 fixed null pointer error when path method is skipped BoykaFramework/boyka-framework
2 participants
@WasiqB @mfaisalkhatri