Skip to content

Feature Matrix documentation - #104

Merged
barriehadfield merged 3 commits into
hyperstack-org:edgefrom
Tim-Blokdijk:edge
Jan 15, 2019
Merged

barriehadfield merged 3 commits into
hyperstack-org:edgefrom
Tim-Blokdijk:edge

Conversation

@Tim-Blokdijk

Copy link
Copy Markdown
Contributor

Based on a short discussion in Gitter chat.
A table listing implemented, partially implemented, missing or bugged features.
Just so we can have some sort of overview what still needs work.

This is just an initial proposal, other people will have to add information. Also, the table columns can be changed.

@barriehadfield
barriehadfield merged commit c0992bb into hyperstack-org:edge Jan 15, 2019
mpantel pushed a commit to mpantel/hyperstack that referenced this pull request Sep 2, 2026
mpantel added a commit to mpantel/hyperstack that referenced this pull request Sep 2, 2026
…yperstack-org#104)

hyperstack-org#44 applied PERMITTED_YAML_CLASSES only on Rails >= 7.1; below that the column
fell back to a bare serialize :data. But Rails 6.1.7.x carries the same
safe-load backport, so YAMLColumn#yaml_load there is already a restricted
safe_load with nothing permitted on this column -- the payload dumps fine and
raises on the way back in. hyperstack-org#44 fixed the dump side; this is the load side.

Unreachable until now: per hyperstack-org#103 the hyperstack_queued_messages table was never
created on Rails 6.1, so nothing was ever queued there and nothing read back.
The two defects were nested -- fixing hyperstack-org#103 turned the path on, and this was
underneath.

Fix applies the same list through a coder on the column (Rails 6.1's serialize
takes any object responding to dump/load) rather than widening
config.active_record.yaml_column_permitted_classes, keeping hyperstack-org#44's principle.
aliases: true because a payload can repeat a value and Psych emits an alias.

Verified across two full-matrix passes (6854, 6856): 20/20 rails-hyperstack cell
runs green. Adds queued_message_coder_spec.rb, including a guard-the-guard
example that fails if Rails stops raising there.

Closes hyperstack-org#104
mpantel added a commit to mpantel/hyperstack that referenced this pull request Sep 2, 2026
hyper-operation: permit the queued-message classes on Rails 6.1 too (hyperstack-org#104)

Closes hyperstack-org#103 and hyperstack-org#104

See merge request ru/hyperstack!143
mpantel added a commit to mpantel/hyperstack that referenced this pull request Sep 2, 2026
mpantel added a commit to mpantel/hyperstack that referenced this pull request Sep 2, 2026
changelog: cover hyperstack-org#104, hyperstack-org#105 and hyperstack-org#100, which landed after the bump commit

See merge request ru/hyperstack!147
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