Comments (12)
May be just adding the dependency in optional
scope will be enough
from shiro.
I am not certain this is a bug. I believe the code started as optional, but later became required, but the optionality was not removed from the code.
IMHO it would be too much of a risk to make it optional at this time, because it would mysteriously break functionality existing applications currently rely on and require inclusions of explicit additional dependency for existing applications.
Perhaps the "action item" is to remove the above reflection / if statement from the code itself?
from shiro.
Maybe someone else can chime in?
from shiro.
I think this is a bug, I will go deeper.
It's well optional on config-ogdl
:
Line 97 in 9507f24
But not in core
:
Line 147 in 9507f24
from shiro.
Maybe not a bug but a code smell? Perhaps the required dependency should be moved into ogdl
and removed from core
but net effect wouldn't change
from shiro.
@lprimak the problem was introduced in this commit:
5755721
Can you explain why adding a provided dependency because of compiler warning?
from shiro.
I remember... it's an OSGi warning from the bnd plugin
from shiro.
I will try a PR to fix it then... thanks for your research @fpapon
from shiro.
The warning from the bnd plugin comes from these lines:
Line 82 in 9507f24
I will make some test in Karaf to see why we need them. I think we don't need to add these import package because we are not using them.
from shiro.
No need. Making it optional in core worked and there is no warning
from shiro.
@papegaaij Thank you for the report! This snuck in during build / code cleanup
from shiro.
No problem. I just happen to notice it because commons-configuration2 triggered a failure due to its dependency on a javax API. Thanks for fixing this!
from shiro.
Related Issues (20)
- [Bug] WildCardPermission.implies() has incorrect check for combination (x,y) permission HOT 2
- shiro-web jakarta only references javax and not jakarta in MANIFEST.MF HOT 7
- [Question] Ldap search filters escape Attributes HOT 1
- How to configure Apache Shiro bundles correctly for use in Apache Karaf HOT 3
- [Question] Having trouble with Jakarta EE and Shiro HOT 14
- [Question] JWT and Stateful HOT 5
- [Bug] Ini.java missing from 1.13.0 shiro-core maven source attachment HOT 2
- [Bug] Caused by: java.lang.NoClassDefFoundError: javax/servlet/Filter HOT 8
- [Enhancement] Jakarta EE9+ without classifiers HOT 2
- [Question] 2.0.1 Release Time? HOT 2
- [Enhancement] Make Shiro compile on JDK 22
- passkey/webauthn support HOT 2
- Wrong class registered in jaxrs feature HOT 2
- [Bug] Cannot login with Shiro2 hashed password from IniRealm HOT 3
- [Enhancement] Migrate integration tests to Jakarta namespace HOT 1
- Bouncycastle dependency problem with Karaf 4.4.5 and Shiro 2.0.0 HOT 1
- How to implement distributed caching HOT 1
- [Enhancement] Jakarta EE module configuration is not flexible enough HOT 1
- [Feature] ShiroFilterFactoryBean with Jakarta Filter compatibility HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from shiro.