Skip to content

Hw38 webflux#16

Open
manfe513 wants to merge 2 commits into
mainfrom
hw38-webflux
Open

Hw38 webflux#16
manfe513 wants to merge 2 commits into
mainfrom
hw38-webflux

Conversation

@manfe513
Copy link
Copy Markdown
Owner

@manfe513 manfe513 commented Jun 3, 2025

Результат на скриншотах:
image
image
image

Васильев Андрей Георгиевич added 2 commits June 3, 2025 20:13
template.convertAndSend(
String.format("%s%s", TOPIC_TEMPLATE, roomId), new Message(HtmlUtils.htmlEscape(message.messageStr())));
switch (roomId) {
case SECRET_ROOM_ID -> logger.info("нельзя писать сообщения в комнату %s".formatted(roomId));
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

тут лучше просто if

.subscribe(message -> template.convertAndSendToUser(principal.getName(), simpDestination, message));
}

private Flux<Message> getMessagesFlux(final long roomId) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Flux в названии лишний.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants