Olog / phoebus-olog

Online logbook for experimental and industrial logging
https://olog.readthedocs.io/en/latest/
Eclipse Public License 1.0
6 stars 13 forks source link

CSSTUDIO-2026: add openapi/swagger + javadoc fix. #184

Closed cjenkscybercom closed 8 months ago

cjenkscybercom commented 8 months ago

Changes:

Fixes:

Screenshot 2024-01-03 at 09 52 55
thelarsjohansson commented 8 months ago

Writing comments after talking to @cjenkscybercom

Good work!

cjenkscybercom commented 8 months ago

Writing comments after talking to @cjenkscybercom

  • I recommend 1.7.0 instead of 1.6.10, as it's most recent release of springdoc
  • below is not necessary, at least not now and for this commit ( try to avoid nice to have )
        <dependency>
            <groupId>org.springdoc</groupId>
            <artifactId>springdoc-openapi-security</artifactId>
            <version>${springdoc.version}</version>
        </dependency>
  • I recommend to have defaults values for Swagger paths and properties in order to have it work in similar ways as to other projects - i.e. not set
springdoc.api-docs.path
springdoc.swagger-ui.path

I also recommend to take care of other application.properties files but this can be done in other commit as those files need oversight anyway.

Good work!

Thank you for your feedback! Fair points, I'll update to the latest 1.x release and remove the security dependency for now.

I do respectfully disagree on using the defaults as they sit in the root path directly (/swagger-ui.html for example); I can see why the CCE team used /api/spec and /api/docs instead since they're in their own path and are easier to remember.

I shouldn't need to apply these changes to the other .properties files though? They inherit from application.properties so I should only need to add overriding or profile-specific properties`?

thelarsjohansson commented 8 months ago

I agree with your comments!

cjenkscybercom commented 8 months ago

I agree with your comments!

Thanks! I've pushed the updates as you suggested, please let me know if you have any other feedback or if you approve (if you have permission to)

cjenkscybercom commented 8 months ago

Thanks Kunal!