Skip to content

Fix Next.js service type check and add configuration logging#169

Open
marcopiraccini wants to merge 7 commits into
mainfrom
more-logs-next-init
Open

Fix Next.js service type check and add configuration logging#169
marcopiraccini wants to merge 7 commits into
mainfrom
more-logs-next-init

Conversation

@marcopiraccini

@marcopiraccini marcopiraccini commented Feb 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Fix: Changed Next.js service type check from 'next' to '@platformatic/next' to match the actual type returned by the runtime config (consistent with other types like @platformatic/service, @platformatic/composer, @platformatic/db)
  • Logging: Added logging during service configuration: service name and type for all services, cache schema detection, and httpCache connection info (host, port, keyPrefix — excluding username/password) for Next.js services
  • Test: Added first test covering @platformatic/next service configuration with cache

@marcopiraccini
Mor logs for next cache setup
c153ecd 
Signed-off-by: marcopiraccini <marco.piraccini@gmail.com>
@marcopiraccini
1.10.1-alpha.0
c3129f1
@marcopiraccini marcopiraccini marked this pull request as draft February 10, 2026 04:18
@marcopiraccini
more logs
4b09603 
Signed-off-by: marcopiraccini <marco.piraccini@gmail.com>
@marcopiraccini
1.10.1-alpha.1
7bb1834
@marcopiraccini
use correct type for next application
bbb8780
@marcopiraccini marcopiraccini changed the title More logs for next cache setup Fix Next.js service type check and add configuration logging Feb 10, 2026
@marcopiraccini marcopiraccini marked this pull request as ready for review February 10, 2026 05:18
@marcopiraccini
version fix
78b11e8 
Signed-off-by: marcopiraccini <marco.piraccini@gmail.com>
@marcopiraccini
fixup
f65a270 
Signed-off-by: marcopiraccini <marco.piraccini@gmail.com>
ivan-tymoshenko
ivan-tymoshenko approved these changes Feb 10, 2026
View reviewed changes

@ivan-tymoshenko ivan-tymoshenko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ivan-tymoshenko ivan-tymoshenko ivan-tymoshenko approved these changes
@ShogunPanda ShogunPanda Awaiting requested review from ShogunPanda

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@marcopiraccini @ivan-tymoshenko