Lua Error for BetterBags when Plugin is enabled

When the plugin gearfirst is enabled, BetterBags throws an error during login or /rl
Message: Interface/AddOns/BetterBags/frames/grid.lua:161: invalid order function for sorting
Time: Sat Jul 27 20:23:45 2024
Count: 1
Stack: Interface/AddOns/BetterBags/frames/grid.lua:161: invalid order function for sorting
[string "@Interface/AddOns/BetterBags/frames/grid.lua"]:161: in function Sort' [string "@Interface/AddOns/BetterBags/views/gridview.lua"]:247: in function Render'
[string "@Interface/AddOns/BetterBags/frames/bag.lua"]:258: in function Draw' [string "@Interface/AddOns/BetterBags/core/init.lua"]:257: in function cb'
[string "@Interface/AddOns/BetterBags/core/events.lua"]:50: in function <Interface/AddOns/BetterBags/core/events.lua:45>
[string "=[C]"]: ?
[string "@Interface/AddOns/ElvUI_Libraries/Core/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:19: in function <...ies/Core/CallbackHandler-1.0/CallbackHandler-1.0.lua:15>
[string "@Interface/AddOns/ElvUI_Libraries/Core/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:54: in function `SendMessage'
[string "@Interface/AddOns/BetterBags/core/events.lua"]:186: in function <Interface/AddOns/BetterBags/core/events.lua:185>
Locals: self =  {
view =  {
}
columns =  {
}
sortVertical = true
spacing = 4
cells =  {
}
maxCellWidth = 2
box = BetterBagsScrollGrid1 {
}
headers =  {
}
cellToColumn =  {
}
idToCell =  {
}
compactStyle = 0
inner = Frame {
}
frame = BetterBagsScrollGrid1 {
}
cellToID =  {
}
bar = Frame {
}
}

`Date: 2024-08-03 16:55:56
ID: 1
Error occured in: Global
Count: 1
Message: Interface/AddOns/BetterBags/frames/grid.lua line 163:
invalid order function for sorting
Debug:
[string "=[C]"]: sort()
[string "@Interface/AddOns/BetterBags/frames/grid.lua"]:163: Sort()
[string "@Interface/AddOns/BetterBags/views/gridview.lua"]:264: Render()
[string "@Interface/AddOns/BetterBags/frames/bag.lua"]:302: Draw()
[string "@Interface/AddOns/BetterBags/core/init.lua"]:258: cb()
[string "@Interface/AddOns/BetterBags/core/events.lua"]:50:
Interface/AddOns/BetterBags/core/events.lua:45
[string "=[C]"]: ?
[string "@Interface/AddOns/AdvancedInterfaceOptions/libs/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:19:
...ons/libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:15
[string "@Interface/AddOns/AdvancedInterfaceOptions/libs/CallbackHandler-1.0/CallbackHandler-1.0.lua"]:54: SendMessage()
[string "@Interface/AddOns/BetterBags/core/events.lua"]:186:
Interface/AddOns/BetterBags/core/events.lua:185
Locals:
(*temporary) =  {
1 =  {
}
2 =  {
}
3 =  {
}
4 =  {
}
5 =  {
}
6 =  {
}
7 =  {
}
8 =  {
}
9 =  {
}
10 =  {
}
11 =  {
}
12 =  {
}
13 =  {
}
14 =  {
}
15 =  {
}
16 =  {
}
17 =  {
}
18 =  {
}
19 =  {
}
20 =  {
}
21 =  {
}
22 =  {
}
23 =  {
}
24 =  {
}
25 =  {
}
26 =  {
}
27 =  {
}
28 =  {
}
}
(*temporary) =  {
view = 2
headerDisabled = false
title = Button {
}
fillWidth = false
maxItemsPerRow = 7
content =  {
}
frame = Frame {
}
}
(*temporary) =  {
view = 2
headerDisabled = false
title = Button {
}
fillWidth = false
maxItemsPerRow = 7
content =  {
}
frame = Frame {
}
}
(*temporary) =  {
view = 2
headerDisabled = false
title = Button {
}
fillWidth = false
maxItemsPerRow = 7
content =  {
}
frame = Frame {
}
}
AddOns:
Swatter, v<%version%> (<%codename%>)
AddonFactory, v11.0.001
AdvancedInterfaceOptions, v2.0.0
AllTheThings, vTWW-4.0.2
Altoholic, v11.0.001
Auctionator, v11.0.0
Auctioneer, v9.1.BETA.5.17 (OneMawTime)
AuctioneerStatsOverTime, v9.1.BETA.5.17 (OneMawTime)
AuctioneerUtilDealFinder, v9.1.BETA.5.17 (OneMawTime)
AuctioneerUtilValuer, v9.1.BETA.5.17 (OneMawTime)
BattleGroundEnemies, v<IP_ADDRESS>
BetterBags, vv0.1.89
BetterBagsBound, vv1.2.0
BetterBagsDragonflightReputationInsignia, vv1.2.0
BetterBagsGearFirst, vv1.0.8
BetterBagsHearthstones, v1.18
BetterBagsWorldEvents, v1.3.3
BetterFishing, v1.1.24
BigDebuffs, vv29
Capping, vv10.2.12
DataStore, v2024.07.28
DataStoreAchievements, v2024.07.28
DataStoreAgenda, v2024.07.28
DataStoreAuctions, v2024.07.28
DataStoreCharacters, v2024.07.28
DataStoreContainers, v2024.07.28
DataStoreCrafts, v2024.07.28
DataStoreCurrencies, v2024.07.28
DataStoreGarrisons, v2024.07.28
DataStoreInventory, v2024.07.28
DataStoreMails, v2024.07.28
DataStorePets, v2024.07.28
DataStoreQuests, v2024.07.28
DataStoreReputations, v2024.07.28
DataStoreSpells, v2024.07.28
DataStoreStats, v2024.07.28
DataStoreTalents, v2024.07.28
DejaCharacterStats, vR110000r1
Details, v#Details.12827.159
DetailsCompare2, v
DetailsEncounterDetails, v
DialogueUI, v0.3.6
HandyNotes, vv<IP_ADDRESS>
HandyNotesBattleForAzerothTreasures, vv32
HandyNotesBurningCrusade, vv6
HandyNotesDragonflightTreasures, vv60
HandyNotesDragonGlyphs, v0.1.5
HandyNotesHigherDimensionalLearning, vv6
HandyNotesKosumoth, vv9
HandyNotesLegionTreasures, vv35.1
HandyNotesLongForgottenHippogryph, vv12
HandyNotesMistsOfPandariaTreasures, vv33
HandyNotesSecretFish, vv6
HandyNotesShadowlandsTreasures, vv78
HandyNotesStygia, vv110000.0
HandyNotesSuramarLeylines, vv8
HandyNotesSuramarTelemancy, vv13
HandyNotesTreasureHunter, vv35
HandyNotesWitheredArmyTraining, v1
HandyNotesWrathOfTheLichKing, vv14
Informant, v9.1.BETA.5.17 (OneMawTime)
KrowiAchievementFilter, v75.2
LeatrixMaps, v11.0.02
LeatrixPlus, v11.0.02
MasterPlanA, vA
MinimalArchaeology, v11.0.0
Narcissus, v1.6.7
OmniCC, v11.0.0
OPie, vZeta 8a
Plumber, v1.3.3
Postal, vv4.1.8
PremadeGroupsFilter, v5.8.2
RareScanner, v<IP_ADDRESS>
Rarity, v1.0 (r733-release)
RarityOptions, v
SavedInstances, v11.0.2
SpeedyAutoLoot, v2.2.21
Stubby, v9.1.BETA.5.17 (OneMawTime)
TomTom, vv4.0.1-release
VenturePlan, v4.42
WarPlan, v3.26
WeakAuras, v5.15.4
WeakAurasArchive, v5.15.4
ZerethMortisPuzzleHelper, v1.6.13
BlizRuntimeLib v<IP_ADDRESS>939(US)
(ck=a4f)
`
Getting similar issue. While Gear First addon is enabled transitioning zones, reloading UI, and relogging will throw this error. I did not get this error on first login though. Only after going from Dalaran to Searing Gorge via the in game portal for the Radiant Echoes event, then returning to Dalaran did the error first appear.

I got the similar error when I have multiple gear set categories in my bag
I guess there are some wrongs in the sorting method
so I simply add a if statement to the original codes in the main.lua file to fix the problem temporally
if (not(string.find(a.title:GetText(), L:G("Gear") .. ":") and string.find(b.title:GetText(), L:G("Gear") .. ":"))) then
    if string.find(a.title:GetText(), L:G("Gear") .. ":") then return true end
    if string.find(b.title:GetText(), L:G("Gear") .. ":") then return false end
end

not sure if there are other situations that also cause this error

Hi mates.
I was sadly not able to reproduce. Do you still have the issue ?
Could you help me to investigate by try to find the use case in which you have the error ?

Maybe a screenshot of your bag items when it's not blank so i can see all the section (item categories) you have to display.
Trying to find the exact setting that making the error show up


[WIP] [Cosmos] Onboard Cosmos to Rush

Changes in the PR -

Updates to the dependency versions
Moving to rush and updating pnpm-lock file
Deleted package-lock file
Removed prettierrc.json from cosmos
Updates related to api-extractor
Updates to the scripts section in package.json
References - updating azure-cosmos-js to azure-sdk-for-js
Pipelines - (Work in progress - js-cosmos-ci)   -

Added PostIntegrationSteps similar to the PreIntegrationSteps
Added cosmos-additional-steps.yml file for building, executing the samples and running consumer-tests
/cc @danieljurek

Future Steps -

Get the latest commits from azure-cosmos-js
Deprecate azure-cosmos-js repo
Format all the files once the complete migration is done

I think it will be better to separate out the changes for adding the ci.yml file (which got deleted in a previous merge) and the rest of the changes. Because looking at the pipeline, it looks like copy task is copying literally everything. The steps of the yaml file are running for other libraries too, so I doubt it's bcz of the yaml file. In order to get a better clarity of why this is happening let's separate out this PR into smaller PRs with unrelated changes maybe?

https://dev.azure.com/azure-sdk/public/_build/results?buildId=93358&view=logs&j=ca395085-040a-526b-2ce8-bdc85f692774
Ongoing investigation related to the failure in Copy packages step in the Build job with @KarishmaGhiya

Thanks, @mikeharder and @KarishmaGhiya.
Build job succeeds now after Mike's fix - https://github.com/Azure/azure-sdk-for-js/pull/4870


FilterType has incorrect field definitions

Product
Hot Chocolate
Version
13.9.6
Link to minimal reproduction
https://github.com/wietlol-org/HotChocolateIssues/tree/main/HotChocolateIssues.InvalidFilterType
Steps to reproduce
Run the unit tests in the HotChocolateIssues.FilterInputType project.
The test DeleteBooks will fail.
After running the tests, the output is written to the TestFiles folder where the erroneous response is presented.
What is expected?
When creating a query/mutation with input according to the following snippet.
[GraphQLType(typeof(FilterInputType<Book>))]
IDictionary<string, object?> filter

The dictionary should only contain the filter as it was specified by the query/mutation input.
What is actually happening?
Instead of only setting the properties that were provided by the input, the current implementation will incorrectly fill almost all fields with the same data.
Given the following input:
mutation {
    deleteBooks(
        filter: {
            title: {eq: "Test"}
        }
    )
}

The dictionary should only contain an entry for "title", which should be a dictionary only containing an entry for "eq".
Looking at HotChocolate.Types.InputParser.cs line 255, the data seems correct:

But the resulting value is definitely not:

The reason for this is that the field definitions of the filter type do not have correct index values:
(HotChocolate.Types.InputObjectType.Initialization.cs line 126)

While their values are being loaded by index:

This means that the resulting dictionary, instead of the following data:
{
	"title": {
		"eq": "Test"
	}
}

will contain the following data:
{
	"and": {
		"and": "Test",
		"or": null,
		"eq": "Test",
		"neq": "Test",
		"contains": "Test",
		"ncontains": "Test",
		"in": "Test",
		"nin": "Test",
		"startsWith": "Test",
		"nstartsWith": "Test",
		"endsWith": "Test",
		"nendsWith": "Test"
	},
	"or": null,
	"title": {
		"and": "Test",
		"or": null,
		"eq": "Test",
		"neq": "Test",
		"contains": "Test",
		"ncontains": "Test",
		"in": "Test",
		"nin": "Test",
		"startsWith": "Test",
		"nstartsWith": "Test",
		"endsWith": "Test",
		"nendsWith": "Test"
	},
	"author": {
		"and": "Test",
		"or": null,
		"eq": "Test",
		"neq": "Test",
		"contains": "Test",
		"ncontains": "Test",
		"in": "Test",
		"nin": "Test",
		"startsWith": "Test",
		"nstartsWith": "Test",
		"endsWith": "Test",
		"nendsWith": "Test"
	}
}

Relevant log output
No response
Additional context
The use of [UseFiltering] is not a valid alternative as these filter fields need to be used in mutations in different places in the input structure.
A good alternative is if a different type than an IDictionary<string, object?> can be used that is deserialized properly that can be used to filter data (IEnumerables/Lists, IQueryable, etc)
But to the best of my knowledge, there isn't.

https://github.com/ChilliCream/graphql-platform/pull/7212

With the mentioned fix, the structure of the data is fixed, but unfortunately, there is still an issue with the data.
After upgrading to 14.0.0-p.122, the resulting data is now:
{
	"and": null,
	"or": null,
	"title": {
		"and": null,
		"or": null,
		"eq": "Test",
		"neq": null,
		"contains": null,
		"ncontains": null,
		"in": null,
		"nin": null,
		"startsWith": null,
		"nstartsWith": null,
		"endsWith": null,
		"nendsWith": null
	},
	"author": null
}

Where the structure is correct, but all fields are added to the dictionary with their default value.
The default value (null) however does not have the same meaning as when the entry is missing in the database.
For example, when we would filter for a value where the title is null, we would do something like this:
{
	"title": {
		"eq": null
	}
}

But if all the fields would be added to the dictionary, it would create a filter that checks if the value is both equal to null and not equal to null and contains null and not contains null and starts with null, etc.
Looking at the HotChocolate.Types.InputParser.cs line 307, it only gives an array of the field values, which are always filled with default values too.
Therefor, overriding the type with an OnComplete/_createInstance implementation will not suffice because the input is insufficient to figure out what data to create.
The CreateInstance method would require more information, for example the processed array, which tells which fields (by index) were processed, or just resultValue which contains the input value.
Can we get a prerelease version where the InputParser can be overridden (not sealed) and injected somehow into the services or where the InputParser forwards the IValueNode resultValue  to the CreateInstance method allowing the types to use it?
Then I can figure out how the createInstance for the dictionary should work.

Have you considered using IFilterContext instead of the value ... IFilterContext gives you the map exactly like you asked for.

What we need is a patching operation. Something that would allow us to update parts of an object without providing full information of the object (as you would need in a replace operation).
Given that HotChocolate is missing this functionality, we rolled our own.
Given the following models and mutation:
public record User(
    string Id
);

public record UserGroup(
    string Id,
    List<User> Users
);

public class Mutations
{
    public async Task<UserGroup> PatchGroupAsync(
        [Service] IUserRepository repository,
        string id,
        UserGroupPatch patch
    )
    {
        return await repository.PatchGroupByIdAsync(id, patch);
    }
}

We are able to create a mutation like the following:
mutation RemoveUserFromGroup(
    $groupId: String!
    $userId: UserGroupPatchInput!
) {
    patchGroup(
        id: $groupId
        patch: {
            users: {
                removeByFilter: {
                    id: { eq: $userId }
                }
            }
        }
    ) {
        id
        users {
            id
        }
    }
}

This would remove a specific user from the group without having to know the other users that are in the group in the input of the mutation.
The removeByFilter property is a property of type [GraphQlType(typeof(AnyType))] Dictionary<string, object?>? at the  moment, but follows the exact same structure as a hot chocolate filter (as it was designed/based on them).
Given that it is an AnyType, the schema won't provide information of how it should be structured, which is annoying.
We wanted to use the InputFilterType to give it that information, but then it failed to deserialize properly.
If I can get the same information from the IFilterContext, that would be great, but keep in mind that the filters should be in specific places in the input structure, because based on where it is used, it has a different meaning, for example under patch.users.removeByFilter or patch.users.patchByFilter.filter or whatever path they would have in whatever structure our models are.
They cannot be on the root, like how queries with [UseFiltering] would be.
And keep in mind that there can be multiple filters in one request, for example if there is both a removeByFilter and a
patchByFilter operation in one mutation request, they would have their own separate filters on the same type.
If IFilterContext suffices to provide that information, can you elaborate how it should be configured?

The filtering infrastructure is not intended to be used for other purposes such as patching, and we won't be making further changes in this regard.


[security] Adding Flask-Talisman

CATEGORY
Choose one

[ ] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
As discussed in the Meetup on 2019-05-03 a security audit at Airbnb surfaced a few recommendations in terms of security header settings. Specifically they recommended:

Setting the secure flag on session cookies
Support HTTP Strict Transport Security (HSTS)

Reading through the Flask security documentation on security header they mention that the Flask-Talisman package can be used to manage various security headers.
It mentions that a subset of default configuration/options handles:

Sets X-Frame-Options to SAMEORIGIN to avoid clickjacking.
Sets Flask's session cookie to secure, so it will never be set if your application is somehow accessed via a non-secure connection.
strict_transport_security, default True, whether to send HSTS headers.

The first one was already enabled for Superset and the later two addresses the two security concerns mentioned above.  Unless we have dedicated or knowledgable security personnel, using something like Flask-Talisman with a blanket type approach to best practices around security seems like a good approach.
Note it's not apparent whether each Superset installation would want to handle security differently and currently the package is configurable only via the Talisman class or Talisman.init_app(...) options. In the future if we require this to be more configurable we could expose some of these configurations in Superset's configuration or leverage the mutator, i.e.,
# superset/__init__.py

talisman = Talisman()

# superset_config.py

FLASK_APP_MUTATOR = mutator

def mutator(app):
    from superset import talisman

    talisman.init_app(app, force_https=False)

TEST PLAN
CI.
REVIEWERS
to: @betodealmeida @DiggidyDave @dpgaspar @graceguo-supercat @michellethomas @mistercrunch

Codecov Report

Merging #7443 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7443      +/-   ##
==========================================
- Coverage   65.26%   65.26%   -0.01%
==========================================
  Files         430      430
  Lines       21078    21080       +2
  Branches     2338     2338
==========================================
+ Hits        13757    13758       +1
- Misses       7205     7206       +1
  Partials      116      116

Impacted Files
Coverage Δ

superset/views/core.py
72.47% <ø> (-0.07%)
:arrow_down:

superset/config.py
93.78% <100%> (ø)
:arrow_up:

superset/__init__.py
74.62% <100%> (+0.38%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8bb7e0...1e02846. Read the comment docs.

ping @betodealmeida @DiggidyDave

:+1:


feat: match the theme color of status bar in light/dark mode

Is your feature request related to a problem? Please describe.
Now the PWA theme_color is hard-coded to #343541 (according to my search), which makes the status bar doesn't looks like a native app in light mode.

Describe the solution you'd like
I'm not sure but there should be a method to match the color.
Describe alternatives you've considered
N/A
Additional context
See also:
https://developer.mozilla.org/en-US/docs/Web/Progressive_web_apps/How_to/Customize_your_app_colors
https://developer.mozilla.org/en-US/docs/Web/CSS/@media/prefers-color-scheme
https://github.com/elk-zone/elk/issues/941
https://github.com/cheeaun/phanpy/issues/231

Thanks for your quick response! Now it can change color based on light/dark mode.
One nitpick is that the white/black color and the status bar don't exactly match the white/black of the app theme, it would be better if they could match!

The updated implementation delegates setting the colour to the device, further improvement won't be possible unless we overhaul the manifest.json code entirely. Thanks for the feedback!


feat: core.ToContext(ctx, v) for ctx initialization

related #831 #841
this brings a method to check and create Instance for ctx, while may someone needs this. (like me ::)

Hi, I have to revert some of your change: you have created an exported API, which is an unsupported usage.
You shouldn't use
//go:linkname mustToContextForced github.com/v2fly/v2ray-core/v4.mustToContext
func mustToContextForced(ctx context.Context, v *Instance) context.Context

to forcefully export this function into your package to create a context suitable for interact with V2Ray's internal component.
This kind of usage is unsupported and may break at any time.

@xiaokangwang ok. i don't know how to design a func MustToContext (i see there is a func MustFromContext as i created a similar one), and i wait for your opinion on this pr (sorry for no extra comment for this). you can do your way kindly, thanks.

The reason I did so is that V2Ray have so many internal components that may change behavior at any time, you shouldn't depend on them remain unchanged.  You could develop software that interact with them, but it may stop working any time.

@xiaokangwang thanks for your explaination.
maybe there is a purpose to make a public ToContext(ctx, v) for go test. https://github.com/v2fly/v2ray-core/blob/1822504ed84bb0de4c41ca888680a6588dead2c2/app/proxyman/outbound/handler_test.go#L70


bugfixadd default value check for hms catalog with dlf

Proposed changes

dlf is not supported default values, so throw an exception.
The database name needs to be case-insensitive and consistent with hive
Improve error message prompt information

run buildall

run buildall

run compile

run buildall

run external

run buildall


Binance: margin trading

Adds new order flags to send margin order instead of spot one.
To cancel/retrieve margin order either custom order param structures could be used or account type could be included in order ID (if Include_Margin_Account_Type_In_OrderId exchange parameter is set to true)

If this approach is accepted then I would like to add support for margin trading to BinanceStreamingExchange. But I do not see examples where connect method accepts custom product subscription classes. And this is required because there should be two additional subscription types for two additional channels: margin trades and isolated margin trades


ListView does not trigger "Loaded" events of items when Animations are turned off

Describe the bug
The "Loaded" events of items in a ListView are not properly triggered when "Show animations" is turned off. (Go to Settings - > Ease to Access -> turn off Show Animations)
Consider a ListView that is bound to items using a data template
        <DataTemplate x:Name="itemTemplate" x:DataType="local:Item">
                <StackPanel Loaded="Grid_Loaded">
                    <TextBox
                        Loaded="TextBox_Loaded"
                        Text="{Binding Path=Value1, Mode=TwoWay}" />

                    <DatePicker
                        Loaded="DatePicker_Loaded"
                        SelectedDate="{Binding Path=Value2, Mode=TwoWay}" />
                </StackPanel>
            </DataTemplate>

<ListView
            ItemTemplate="{StaticResource itemTemplate}"
            ItemsSource="{Binding Children}"/>

When an item is added the first time, the Loaded-events in the template (e.g. the event on the StackPanel, or the two other on the TextBox or DatePicker) are properly invoked.
When removing an item in the bound collection, the loaded event is also invoked. This seems to be wrong - when removing an item, the Loaded event should not be triggered.
When adding a new item again to the list, the Loaded are not triggered at all for this item. This seems to be a bug.
Steps to reproduce the bug

Go to the settings -> Ease to Access and turn off the Show Animations in Windows.
Rebuild and run the attached project.
Click the Add button.
Loaded-events will be called for the Textbox and Datepicker, which is good
Click the Remove button for the freshly added item.
Loaded events are called again for the Textbox and Datepicker - this is unexpected.
Again click add button.
Now the Loaded  events are not triggered at all

Expected behavior
When removing an item from the ItemSource of a ListView, the Loaded events should not be triggered.
When re-adding items again, the Loaded events should be triggered.
Screenshots
LoadedIssueinListView.zip
NuGet package version
WinUI 3 - Windows App SDK 1.4.0: 1.4.230822000
Windows version
Windows 11 (22H2): Build 22621, Windows 11 (21H2): Build 22000, Windows 10 (21H2): Build 19044, Windows 10 (21H1): Build 19043, Windows 10 (20H2): Build 19042
Additional context
No response

Just in case somebody stumbles across the same issue. I create a workaround for this, by creating always a new data template (with XamlReader.Load(..)) when re-adding an item. The new data template has a different name, which seem to convince the ListView to properly call the Loaded-event again. Posting this here because it looks like the WinUI team is currently swamped with issues.
ListViewIssue_SolutionWithUserControl.zip

Technically Loaded and Unloaded events should fire whenever the UI element is added to the visual tree. Bug you describe is probably related to the UI virtualization. This functionality re-uses the same UI elements but only updating the bindings on the fly. In that case this should not be considered as a bug, because you are re-using the existing container that is already realized in the visual tree.
If you disable virtualization by using a different ItemsPanelTemplate (like a Stackpanel) in your ListView, this issue might not happen.
This is just a theory though, I'm pretty sure WinUI guys can investigate the scenario better here :)

Hi @bkaankose ,
thanks a lot for pointing this out! I just tried it out:
<ListView.ItemsPanel>
    <ItemsPanelTemplate>
        <StackPanel Orientation="Vertical" />
    </ItemsPanelTemplate>
</ListView.ItemsPanel>

and now everything works fine!
Looks like the current ItemsPanelTemplate (which is <ItemsStackPanel Orientation="Vertical" />  in generic.xaml) seems to cause the issue.
Looks like the best workaround for the time being.

@andreaslennartz this is by design. ListView virtualizes its items as @bkaankose mentions above and loaded will not be triggered when items are recycled. Use ContainerContentChanging or other events to update the container when items come into view or when they get recycled.


Add CBMC proof of Free_FreeRTOS function

Add CBMC proof of Free_FreeRTOS function
Description
This PR adds files required for the CBMC proof of free_FreeRTOS function.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is formatted using Uncrustify.
[x] I have read and applied the rules stated in CONTRIBUTING.md.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

What version of the kernel is the submodule set to? It should be on the latest release and not on master.

Also, maybe try allocating an actual pointer (using malloc) and then free it as well.

Also, maybe try allocating an actual pointer (using malloc) and then free it as well.

Since we are not checking the memory safety of the external library functions. It does not make sense to do this.


Run cred rotation every month

Jenkins recently went down for a while because our creds expired. Running the IODatastores rotation job fixed it. Given that its relatively cheap, we should probably run that job every month so that we're not cutting it close on expirations. I could actually be convinced that running it even more frequently is a good idea.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@eilnk21 are there any reasons that you know of not to run this more frequently? (asking since you added this in https://github.com/apache/beam/pull/17383)

@elink21 are there any reasons that you know of not to run this more frequently? (asking since you added this in https://github.com/apache/beam/pull/17383)

@elink21 are there any reasons that you know of not to run this more frequently? (asking since you added this in #17383)

The initial bi-monthly frequency was proposed given the credentials lifetime reported by this command. As all the jobs get their cluster credentials dynamically through gcloud, I think there won't be a huge impact if those are rotated on a more frequent basis.

R: @Abacn

Thanks. Is the original every other month setting never working?

It ran for the first time a little less than 2 months ago, but the creds expired before the second run. Had there been a run in between we would've most likely avoided the problem


why can not i delete object?? [Can only delete objects from the Realm they belong to.]

I wanna delete objects in realm.
My Model class
class CardInformation: Object {
    dynamic var bankName = ""
    dynamic var cardImage : NSData?

    override class func primaryKey() -> String {
        return "bankName"
    }
}

My TableViewController
class CardListTableViewController : UITableViewController {
    var realm = try! Realm()
    var results : Results<CardInformation>?
    var cardInfo : CardInformation?

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let objects = results?[indexPath.row]
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell")
            cell?.textLabel?.text = objects?.bankName

        return cell!
    }

    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        cardInfo = CardInformation()
        cardInfo?.bankName = (results?[indexPath.row].bankName)!
        cardInfo?.cardImage = results?[indexPath.row].cardImage
        performSegue(withIdentifier: "segue_insert", sender: self)
    }

    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCellEditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == .delete {
            realm.beginWrite()

           //1. Can only delete objects from the Realm they belong to.
            realm.delete(CardInformation(value: ["bankName":(results?[indexPath.row].bankName)!, "cardImage":(results?[indexPath.row].cardImage)!] as [String: Any]))

//or
            //2. Can only delete objects from the Realm they belong to.
            realm.delete((results?[indexPath.row])!)

//or
            //3. Can only delete objects from the Realm they belong to.
            let deleteItem = results?.filter("bankName == %@", (results?[indexPath.row].bankName)!)
            realm.delete(deleteItem!)

            try! realm.commitWrite()
            tableView.reloadData()
        }
    }
}

Goals
What do you want to achieve?
Expected Results
What did you expected to happen?
Actual Results
What did happened instead?
e.g. the stack trace of a crash
Steps to Reproduce
What are steps we can follow to reproduce this issue?
Code Sample
Provide a code sample or test case that highlights the issue.
If relevant, include your model definitions.
For larger code samples, links to external gists/repositories are preferred.
Alternatively share confidentially via mail to<EMAIL_ADDRESS>Full Xcode projects that we can compile ourselves are ideal!
Version of Realm and Tooling
In the CONTRIBUTING guidelines, you will find a script,
which will help determining these versions.
Realm version: 2.0.2
Xcode version: 8.0
iOS/OSX version: 10.0.2
Dependency manager + version: ?

Hi there! Thanks for the issue!
Just like the error states, you can only delete Object model objects from the Realm in which they were originally saved. Judging from your code, that would seem to indicate that however you are querying for your results object, they're coming from a different Realm than the one that is provided with the default Realm() call.
One easy way to fix this is to use results?.realm to explicitly reference the origin Realm from where it came.
Finally, just to point it out, realm.delete(CardInformation()) won't work because it's creating a new instance of CardInformation that wouldn't have been added to a Realm yet.
I hope that answered your question! Let me know if you have any more!

oh i really appreciate that! Thank you!!


OCI documenation updates

Required items, please complete
Change(s):

Updates for OCI documenation

Reason for Change(s):

Customers facing issues in Oracle and sentinel side
Resolves ISSUE #9203

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updates for OCI documenation

Reason for Change(s):

Customers facing issues in Oracle and sentinel side
Resolves ISSUE #9203

Version updated:

No
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Closing PR


New release

There are some interesting changes in master, it'd be nice if a new tag/release came for downstream distros to pick up.

In the following days I'll merge some more stuff and make a new release.

Thank you!


Using OpenWrt 15.05.1 can't login when LAN set to 192.168.0.x

So, plug in laptop to LAN port, gets a 192.168.1.x address. Webpage to <IP_ADDRESS>. Show Luci text then redirects to login screen. Click Login and get in. Change the LAN static IP address from <IP_ADDRESS> to <IP_ADDRESS> (to sit with my existing LAN), click Save and Apply. Lose the webpage (to be expected), from my laptop renew IP, get a 192.168.0.x address, open webpage to <IP_ADDRESS> and it just sits on the LUCI text screen. Click on that and get the login screen. Click Login and it rejects it. I've also tried by setting a password before changing the IP address. Same result.
If I set the IP address to <IP_ADDRESS> or .3.1 etc it's fine. Is this a 'feature' that I should know about?

Well, new day, new issues !!! It now logs in (but slowly) but after changing any setting (normally WiFi based), I get logged out :-0

Sounds like you have an IP address conflict or something in your LAN. There is no special handling of <IP_ADDRESS> anywhere in OpenWrt, LEDE or LuCI.


Logging every REST API call

I'm running stargate as a systemd process and running into an issue with every REST API call being logged to /var/log/messages. I have configured the logback.xml to send output to a file, and I see logs in that file. Interestingly, all the logs seem to be coming from "starctl", which I believe has exited after starting the stargate java server.
In a production environment, I don't think it makes sense to log every REST API call, so the goal here would be to stop those logs from showing up in any output.
Any ideas?
==========================================================
Here's an example of a log message going to /var/log/messages and not going into the configured logback file:

Apr 26 15:25:20 localhost starctl: <IP_ADDRESS> - - [26/Apr/2021:21:25:20 +0000] "POST /v2/keyspaces/DPI/Stats5S HTTP/1.1" 201 68 "-" "Apache-HttpClient/4.5.13 (Java/1.8.0_272)" 37

Here's the contents of the logback.xml file:

    <!-- For Assistance related to logback-translator or configuration  -->
    <!-- files in general, please contact the logback user mailing list -->
    <!-- at http://www.qos.ch/mailman/listinfo/logback-user             -->
    <!--                                                                -->
    <!-- For professional support please see                            -->
    <!--    http://www.qos.ch/shop/products/professionalSupport         -->
    <!--                                                                -->
    <configuration>
       <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
          <!--See also http://logback.qos.ch/manual/appenders.html#RollingFileAppender-->
          <file>/var/log/probe/cassandra/stargate.log</file>
          <rollingPolicy class="ch.qos.logback.core.rolling.FixedWindowRollingPolicy">
             <fileNamePattern>/var/log/probe/cassandra/stargate.%i.gz</fileNamePattern>
             <minIndex>1</minIndex>
             <maxIndex>3</maxIndex>
          </rollingPolicy>
          <triggeringPolicy class="ch.qos.logback.core.rolling.SizeBasedTriggeringPolicy">
             <maxFileSize>20MB</maxFileSize>
          </triggeringPolicy>
          <encoder>
             <pattern>%date [%thread] %-5level %logger{35} - %msg%n</pattern>
          </encoder>
       </appender>
       <root level="INFO">
          <appender-ref ref="FILE" />
       </root>
    </configuration>

What stargate version are you currently running? This PR helped improve our access log config and was added in v1.0.18
https://github.com/stargate/stargate/pull/816

We're using 1.0.15. I'll pull your latest and give it a try.
I'll close this for now.


Copy-FileToBcContainer fails for BcContainerHelper 6.0.16

While creating our docker containers we import external DLLs to the container via Copy-FileToBcContainer
This worked fine until BcContainerHelper 6.0.16.
The script:
    Write-Host "Install third-party DLLs" -ForegroundColor Blue
    $ThirdPartyDll | ForEach-Object {
        Write-Host "Install $($_.Name)"
        $TargetDll = Join-Path -Path $AddInsPath -ChildPath $_.Name
        Copy-FileToBCContainer `
            -containerName $containerName `
            -localPath $_.FullName `
            -containerPath $TargetDll
    }

Output for BcContainerHelper 6.0.15:
Check for TBarCode11_x64.dll
Install third-party DLLs
Install TBarCode11_x64.dll
Copy C:\a\1\s\scripts\files\TBarCode11_x64.dll to container industry-136782-onprem (C:\Program Files\Microsoft Dynamics NAV\240\Service\Add-ins\TBarCode11_x64.dll)
...next install...

Output for BcContainerHelper 6.0.16:
Check for TBarCode11_x64.dll
Install third-party DLLs
Install TBarCode11_x64.dll
Copy C:\a\1\s\scripts\files\TBarCode11_x64.dll to container industry-137282-onprem (C:\Program Files\Microsoft Dynamics NAV\240\Service\Add-ins\TBarCode11_x64.dll)
Cannot find path 'C:\ProgramData\BcContainerHelper\b7fcd4ef-ab2c-4b70-a0b2-2523127dfaf1' because it does not exist.

I cannot repro this
I would need the full output from when you created the container, which includes OS versions, PowerShell versions etc etc.

Shipped in preview


A Few More Order Types - Interactive Brokers

New order type for IB - VWAP.

All or None // Fill or Kill Option on Orders

Also would be nice to have Immediate or Cancel (IOC):
https://www.interactivebrokers.com/en/index.php?f=594

Adaptive Algo
https://interactivebrokers.github.io/tws-api/ibalgos.html#gsc.tab=0

Any update on these features? can we at least have all-or-none for now? Thanks!

Not yet sorry but we'll reply here when it is done so no need to ping thread.
Its an internal rule of QC -- bugs before features :) As long as there are red-flags we'll fix them first. If you'd like to push these features through we'd welcome a PR though!

Bugs before Features is a good policy. I really need this feature because my algo goes crazy because of the partial fills - so much that I pulled the plug on it today.
I will see if I can contribute. Otherwise looks like I have to wait for you guys to pick it up.

Extra comments/notes here: https://github.com/QuantConnect/Lean/pull/1203

There are multiple features grouped together in this issue, so we are splitting this issue in the following tasks, to keep pull requests smaller and easier to review:

[ ] Rename enum OrderDuration -> TimeInForce and rename Order.Duration to Order.TimeInForce

[ ] Implement TimeInForce.Day (DAY)
https://www.interactivebrokers.com/en/software/tws/usersguidebook/ordertypes/time_in_force_for_orders.htm

[ ] Implement TimeInForce.GoodTilDate (GTD)
https://www.interactivebrokers.com/en/index.php?f=589

[ ] Implement TimeInForce.FillOrKill (FOK)
https://www.interactivebrokers.com/en/index.php?f=586

[ ] Implement TimeInForce.ImmediateOrCancel (IOC)
https://www.interactivebrokers.com/en/index.php?f=594

[ ] Implement Order.AllOrNone (AON)
https://www.interactivebrokers.com/en/index.php?f=577

[ ] Implement IB Algorithms with InteractiveBrokersOrderProperties
https://interactivebrokers.github.io/tws-api/algos.html

@StefanoRaggi  --  a reminder that for each step we're only implementing them for interactive brokers, which means for all other brokerage models, we'll need to update the CanSubmit method to flag orders with these options as invalid.

Based on many user requests, it would be nice to implement OCA - One Cancels All grouped orders. https://www.interactivebrokers.com/en/index.php?f=617

Combination orders

Relative Order type:
https://www.interactivebrokers.com/en/general/education/videos/tws-python-api-pair-trades.php


On zsh, goto is unable to create aliases if ~/.goto does not exist

zsh version: zsh 5.5.1 (x86_64-redhat-linux-gnu) with Prezto
goto version: https://github.com/iridakos/goto/commit/cd760f27f2d5133343f2e24eaedab1133ba60516

If ~/.goto does not exist, goto is unable to create aliases. It will print an error, claim success but will not effectively save anything:
~/Documents/Git/godotengine/godot
❯ goto -r godot .
_goto_register_alias:34: no such file or directory: /home/hugo/.goto
Alias 'godot' registered successfully.

~/Documents/Git/godotengine/godot
❯ goto -l
You haven't configured any directory aliases yet.

~/Documents/Git/godotengine/godot
❯ goto godot
goto error: unregistered alias godot

This can be worked around by entering touch ~/.goto. Note that this only applies to zsh — it seems to work fine with Bash when ~/.goto does not exist (i.e. the file will be created automatically).

Note that in ZSH there is an excellent and very similar tool called wd which does exactly what goto does
For ZSH / oh-my-zsh: https://github.com/mfaerevaag/wd
For any shell: https://github.com/mfaerevaag/wd-c


5.23.2 includes deprecation without replacement

Description
When updating to v5.23.2 we found that setBreadcrumbCapacity: is deprecated for setMaxBreadcrumbs: but setMaxBreadcrumbs: is not in the BugsnagConfiguration header file yet, only the implementation file.
Issue
This deprecation exists https://github.com/bugsnag/bugsnag-cocoa/blob/release-v5.23.2/Source/Bugsnag.h#L274-L275 but this header https://github.com/bugsnag/bugsnag-cocoa/blob/release-v5.23.2/Source/BugsnagConfiguration.h doesn't have the new function. This blocks updates if you use warnings as errors
Environment
Library versions: v5.23.2

Hi @keith, thanks for the report. We'll take a look into this.

Hey @keith, the maxBreadcrumbs property is declared here https://github.com/bugsnag/bugsnag-cocoa/blob/release-v5.23.2/Source/BugsnagConfiguration.h#L253 which synthesizes the getters and setters.
Testing this with v5.23.2 I see no issues when using the new method (setMaxBreadcrumbs), and a warning (or error if you've enabled warnings as errors) if I use setBreadcrumbCapacity.
Can you please let us know some more details of what you see if you try using the new method?

Ah sorry I took the conversion instructions too literally and tried to use configuration.setMaxBreadcrumbs(100) in Swift instead of configuration.maxBreadcrumbs = 100


[Feature request] Support for eslint v9.1.1

Is your feature request related to a problem? Please describe.
I'm using nuxt and the latest @nuxt/eslint module which utilizes eslint version 9 and I couldn't make it work.
If I import the eslint-plugin-tailwindcss index file in the new flat-config syntax e.g. an eslint.config.mjs, I'll get an error:
[Error - 13:47:05] Request textDocument/formatting failed.
  Message: Request textDocument/formatting failed with message: Config (unnamed): Key "rules": Key "classnames-order": Expected severity of "off", 0, "warn", 1, "error", or 2.
  Code: -32603

And here the config file (eslint.config.mjs):
// @ts-check
import withNuxt from './.nuxt/eslint.config.mjs'
import pluginVue from 'eslint-plugin-vue'
import pluginTailwindCSS from 'eslint-plugin-tailwindcss'

export default withNuxt([
  ...pluginVue.configs['flat/recommended'],
  pluginTailwindCSS,
  {
    rules: {
      // ...
    }
  }
])

Describe the solution you'd like
My guess is to update to the new flat-config syntax or at least describe in the documentation how to properly implement it for the new configuration syntax.

using eslint v8 (which also uses the flat-config), you can do this:
// @ts-check
import { FlatCompat } from '@eslint/eslintrc'
import withNuxt from './.nuxt/eslint.config.mjs'

const compat = new FlatCompat()

export default withNuxt(
	{
		rules: {
			// ...
		}
	}, ...compat.config({
		extends: ['plugin:tailwindcss/recommended'],
		rules: {
			'tailwindcss/no-custom-classname': 'off',
			'tailwindcss/migration-from-tailwind-2': 'off',
		},
	})
)

this doesn't seem to work with v9 anymore however. i'd recomment sticking to v8 for now.

Regarding eslint v9 support:
This comment describes the current source of incompatibility:
https://github.com/eslint/eslint/issues/18405#issuecomment-2084982903
It's the usage of the deprecated context.parserServices property.

Greetings @francoismassart. I installed the 3.15.3-beta.9 and it appears that every single rule is detecting errors where it should, the plugin is working as expected.
That said, I am not sure if this is intended or not, but the recommended config is still using the eslintrc format, rather than the flat config format. So for example, a config like this won't work out of the box:
import eslintPluginTailwindCss from 'eslint-plugin-tailwindcss';

export const eslintPluginTailwindCssConfig = [
    {
        ...eslintPluginTailwindCss.configs.recommended,
    }
];

For me this is not an issue because of how I configure the plugin, but I thought it might be worth mentioning anyway.
Cheers!

I've tried it like that:
import js from "@eslint/js";
import ts from "typescript-eslint";
import tailwind from "eslint-plugin-tailwindcss";

export default [
  js.configs.recommended,
  ...ts.configs.recommended,
  ...tailwind.configs["flat/recommended"],
];

It seems, that the 3.15.3-beta.9 version in NPM isn't updated, because the downloaded code is still missing the flat/recommended node in the config, as you can see here in the lib/index.js opposing to the lib/index.js in the GH repos.
It is working if I set it up like this:
//...
  {
    plugins: {
      tailwindcss: pluginTailwindCSS,
    },
    rules: {
      'tailwindcss/classnames-order': 'warn',
      'tailwindcss/enforces-negative-arbitrary-values': 'warn',
      'tailwindcss/enforces-shorthand': 'warn',
      'tailwindcss/migration-from-tailwind-2': 'warn',
      'tailwindcss/no-arbitrary-value': 'warn', // this is disabled in the recommended config
      'tailwindcss/no-custom-classname': 'warn',
      'tailwindcss/no-contradicting-classname': 'error',
      'tailwindcss/no-unnecessary-arbitrary-value': 'warn',
    },
  },
//...

Here is a new version to test,
I need feedback from user using:

ESLint < v9 + legacy config
ESLint < v9 + flat config
ESLint v9 + legacy config
ESLint v9 + flat config

The beta version to be tested is<EMAIL_ADDRESS>which can be installed via
npm i<EMAIL_ADDRESS>-D

Thanks for the update.
I've tested it with ESLint v9 + flat config and it seems to work just fine.
Here is my eslint.config.mjs:
//...
import eslintPluginTailwindCSS from 'eslint-plugin-tailwindcss'
import withNuxt from './.nuxt/eslint.config.mjs'

export default withNuxt([
  //...
  ...eslintPluginTailwindCSS.configs['flat/recommended'],
  //...
])

Working fine with ESLint 8.57.0 + flat config + Nuxt + AntFu.
eslint.config.js:
import antfu from '@antfu/eslint-config'
import eslintPluginTailwindCSS from 'eslint-plugin-tailwindcss'
import withNuxt from './.nuxt/eslint.config.mjs'
//...

export default withNuxt(
  antfu(
    //...
    ...eslintPluginTailwindCSS.configs['flat/recommended'],
    {
      rules: {},
    },
  ),
)

Working fine with ESLint 9.3.0 + Flat config repo link
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-expect-error
import pluginTailwindcss from 'eslint-plugin-tailwindcss'
import type { Linter } from 'eslint'

export default function tailwindcss(): Linter.FlatConfig[] {
  return [
    {
      name: 'tailwindcss/rules',
      plugins: {
        tailwindcss: pluginTailwindcss
      },
      rules: {
        'tailwindcss/classnames-order': 'error',
        'tailwindcss/no-contradicting-classname': 'error',
        'tailwindcss/enforces-shorthand': 'error',
        'tailwindcss/no-custom-classname': 'off',
        'tailwindcss/no-unnecessary-arbitrary-value': 'off',
      }
    }
  ]
}

ESLint v9 + flat config +<EMAIL_ADDRESS>all working good. Repo

Latest version (3.16.0) support both flat config and ESLint 9
npm i<EMAIL_ADDRESS>-D

@francoismassart Thanks for your work!
Okay, I tested it and had a little hiccup with the different versions 3.16.0-beta.0 & 3.15.3-beta.9 but 3.15.3-beta.9 now seems to work if I use it like this:
import pluginTailwindCSS from 'eslint-plugin-tailwindcss'
import withNuxt from './.nuxt/eslint.config.mjs'

export default withNuxt([
  {
    plugins: { tailwindcss: pluginTailwindCSS },
    rules: pluginTailwindCSS.configs.recommended.rules,
  },
  // ...
])

This is the way it is intended to be, right?

Thank you! This config also work for me.


update(Docker): use prod dependencies only

yarn install --production --frozen-lockfile is somehow equivalent to npm prune
Before
Final docker image size was 310MB 👎
dockerize job takes 2m:59s 👍
insecure ❌
After
Final docker image size becomes 179 MB 👍
dockerize job takes 3m:43s 👎
more secured ✅

Fixes #5


chore: bdd-nh-1586997820 to 0.0.1

chore: Promote bdd-nh-1586997820 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Wrong Order Details, when the order has an error

Describe the bug
When an order is made, sometimes the shop does not process the transaction further, we only see a loading animation. When this happens we see that the function "ajaxDoPayment" results in an error where the OrderID creates a mysql-error. When this happens the Customer-Data is altered by adyen. The username and also his addresses are completely different, it seems as there were different data loaded from another shop-system or Adyen customer. If this is true and these data are not some kind of random Userdata or testdata, this will also be a security issue, as i now have the customer data of someone elses shop-system.
Expected behavior
The order gets processed without an error and the customer data is not altered.
Desktop (please complete the following information):

OS: OSX
Browser Chrome
Version 88

Additional context
Shopversion: 5.6.9
Pluginversion: 1.6.0 (from Shopware Store)
Plugin is running in Test-Mode.
Error Message:

Oeps! Er is een fout opgetreden!  De hiernavolgende informatie helpt u wellicht verder.
It is a very strange behavior indeed. Could you provide some detailed steps to try to reproduce?
Thanks,
Ángel

Hello @danielDigitalArt, thanks for opening this issue.
It is a very strange behavior indeed. Could you provide some detailed steps to try to reproduce?
Thanks,
Ángel

Sadly not really, all what happens is that i get an error when i try to get to the Payment Page of the selected Adyen Payment, i had this with multiple (ideal) Payments like Test Issuer 5 etc.
I dont get to the proper Payment page as the Shop is stuck with that loading animation in the frontend, while showing the added error in the browsers console.
So i can select i.e. Idead ° Test Issuer 5 as payment method, then i want to go to the last step, where i get redirected to the adyen payment page, but before getting there the error happens.

Sadly not really, all what happens is that i get an error when i try to get to the Payment Page of the selected Adyen Payment, i had this with multiple (ideal) Payments like Test Issuer 5 etc.
I dont get to the proper Payment page as the Shop is stuck with that loading animation in the frontend, while showing the added error in the browsers console.
So i can select i.e. Idead ° Test Issuer 5 as payment method, then i want to go to the last step, where i get redirected to the adyen payment page, but before getting there the error happens.

@acampos1916 i think i could rule out the problem here now. The hint was the duplicate entry, there were some Data from Testings and i guess from imports, the data was in the order_billingaddress and was then added to the current orderid. I do not really know why the orderid was reset, but it explains Userdata appearing out of nowhere. So i tested after cleaning the Tables and could not yet see this particular Problem again, also the duplicate error Message in general should be gone away too now.


Build and ship kbfsfuse too

@oconnor663

These commits are included in https://github.com/keybase/client/pull/1689.


Comp4711 Tutorial02c

Continuing Lab
Signed-off-by:uoi100<EMAIL_ADDRESS>

Not finished yet


[sentry][web] Refused to set unsafe header "User-Agent"

not related to flutter but flutter/sentry does not have is own issue tracker.
When using the package on a web application and reporting an error, I am getting this error.
html_dart2js.dart:22938 Refused to set unsafe header "User-Agent".
But I can see the issue on the dashboard.
I just hope it is not causing this https://github.com/flutter/flutter/issues/16883

That is not allowed and prevented by the browser AFAIK

linked to https://github.com/flutter/flutter/issues/16883


Multiple additions

I have added a feature of adding multiple arguments experimentally for adding new tasks .

Please ignore the update README.md commit . That was an accident.

If you are fine with this , I can extend this feature to other commands also.


PutParametersInSession with xpathExpression with type=domdoc results in "Content is not allowed in prolog"

Describe the issue
After storing (a part of) an XML in a session key using a PutParametersInSession and type="domdoc", further xpathExpressions on that session key result in "Content is not allowed in prolog" errors.
Setting type="xml"  works as intended, but I would expect type="domdoc" to not have different behavior.
The issue is present in 7.5.2, 7.6 and 7.7.
In 7.5.2 the error is Parameter [xmlMessageChild2] exception on transformation to get parametervalue: (ClassCastException) org.apache.xerces.dom.DocumentImpl cannot be cast to java.lang.String"
In 7.6 and 7.7 the error is Parameter [xmlMessageChild2] exception on transformation to get parametervalue: (XPathException): (SAXParseException) line [1] column [1]: Content is not allowed in prolog."
Reporter
Ricardo

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
Configuration
<module>
	<adapter name="Repro" description="Repro">
		<receiver name="Repro"
			className="nl.nn.adapterframework.receivers.GenericReceiver">
			<listener name="Repro"
				className="nl.nn.adapterframework.receivers.JavaListener"/>
		</receiver>

		<pipeline firstPipe="Store">
			<exits>
				<exit path="READY" state="success"/>
			</exits>

			<pipe name="Store"
				className="nl.nn.adapterframework.pipes.PutParametersInSession">
				<param name="xmlMessageChild" xpathExpression="Test/Child[1]" type="domdoc"/>
				<forward name="success" path="Store2" />
			</pipe>

			<pipe name="Store2"
				className="nl.nn.adapterframework.pipes.PutParametersInSession">
				<param name="xmlMessageChild2" sessionKey="xmlMessageChild" xpathExpression="Child/name"/>
				<forward name="success" path="READY" />
			</pipe>
		</pipeline>
	</adapter>
</module>

Input
<?xml version="1.0" encoding="UTF-8"?>
<Test>
	<Child><name>X</name></Child>
	<Child><name>Y</name></Child>
	<Child><name>Z</name></Child>
</Test>

Environment
FF! 7.7-20210928.133035: ibis4repro
Running on LPZWNL000014943 using Apache Tomcat/9.0.50
Java Version: OpenJDK Runtime Environment (1.8.0_292-b10)
Heap size: 261M, total JVM memory: 450M
Free disk space: 58GB, total disk space: 237GB
Up since: 2021-09-28 15:56:47 (14m)

Additional Environment

DBMS: [e.g. Oracle, MSSQL, MariaDB, MySql, PostgreSQL]
Browser: [e.g. Chrome, Safari, Edge, Firefox]

Additional Context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

The difference between dom-doc and xml types should be documented.

Has this ever worked in earlier versions like 7.3?


[Doc Improvement][Update links in Tools and SDK for Visual Studio]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 21ce85c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 03e3d18:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit ad7e452:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 8f998d9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit f72a0ca:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/mstdd-landing.yml
100
0
link
:white_check_mark:

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
95
6
link
:white_check_mark:

msteams-platform/whats-new.md
91
35
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit c5fcb45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/mstdd-landing.yml
100
0
link
:white_check_mark:

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

msteams-platform/toolkit/Provision cloud resources using Visual Studio.md
97
30
link
:white_check_mark:

msteams-platform/toolkit/create-new-teams-app-for-Visual-Studio.md
100
2
link
:white_check_mark:

msteams-platform/toolkit/debug-teams-app-visual-studio.md
98
5
link
:white_check_mark:

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
95
6
link
:white_check_mark:

msteams-platform/whats-new.md
91
35
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit e0f0be7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/toolkit/create-new-teams-app-for-Visual-Studio.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/debug-teams-app-visual-studio.md
:white_check_mark:Succeeded
View

[msteams-platform/toolkit/Provision cloud resources using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/doc-improvement-update-sbs-build-your-first-app-using-csharp/msteams-platform/toolkit/Provision cloud resources using Visual Studio.md)
:white_check_mark:Succeeded
View

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/mstdd-landing.yml
100
0
link
:white_check_mark:

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md
98
2
link
:white_check_mark:

msteams-platform/toolkit/Provision cloud resources using Visual Studio.md
97
30
link
:white_check_mark:

msteams-platform/toolkit/VS-TeamsFx-preview-and-customize-app-manifest.md
100
1
link
:white_check_mark:

msteams-platform/toolkit/create-new-teams-app-for-Visual-Studio.md
100
2
link
:white_check_mark:

msteams-platform/toolkit/debug-teams-app-visual-studio.md
98
5
link
:white_check_mark:

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
95
6
link
:white_check_mark:

msteams-platform/whats-new.md
91
35
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit e5ee016:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-csharp.yml
:white_check_mark:Succeeded
View

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/toolkit/create-new-teams-app-for-Visual-Studio.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/debug-teams-app-visual-studio.md
:white_check_mark:Succeeded
View

[msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/doc-improvement-update-sbs-build-your-first-app-using-csharp/msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md)
:white_check_mark:Succeeded
View

[msteams-platform/toolkit/Provision cloud resources using Visual Studio.md](https://github.com/MicrosoftDocs/msteams-docs/blob/doc-improvement-update-sbs-build-your-first-app-using-csharp/msteams-platform/toolkit/Provision cloud resources using Visual Studio.md)
:white_check_mark:Succeeded
View

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/VS-TeamsFx-preview-and-customize-app-manifest.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/TOC.yml
98
6
link
:white_check_mark:

msteams-platform/mstdd-landing.yml
100
0
link
:white_check_mark:

msteams-platform/sbs-gs-csharp.yml
95
17
link
:white_check_mark:

msteams-platform/toolkit/Deploy Teams app to the cloud using Visual Studio.md
98
2
link
:white_check_mark:

msteams-platform/toolkit/Provision cloud resources using Visual Studio.md
97
30
link
:white_check_mark:

msteams-platform/toolkit/VS-TeamsFx-preview-and-customize-app-manifest.md
100
1
link
:white_check_mark:

msteams-platform/toolkit/create-new-teams-app-for-Visual-Studio.md
100
2
link
:white_check_mark:

msteams-platform/toolkit/debug-teams-app-visual-studio.md
98
5
link
:white_check_mark:

msteams-platform/toolkit/teams-toolkit-overview-visual-studio.md
95
6
link
:white_check_mark:

msteams-platform/whats-new.md
91
35
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Error: Could not spawn setenv

One of my students is getting the attached error. Any ideas?

The student resolved it: it was anti-viral software.


[Feature] OR logic for Filters

Is your feature request related to a problem? Please describe.
I find myself needing an OR operation between filters all the time. It's very hard to select runs otherwise.
Describe the solution you'd like
In the filter screen, if I click checkmarks next to two filters, it does an AND operation between the filters. But for example, I want to find all runs that are "status=running" or "name=dog". This is excruciating to do right now in the current interface.
Describe alternatives you've considered
A simple line of logic to allow OR logic between filters.
Additional context

Add a UI element that enables OR operations here.

Hey @richardrl , thank you for the feature request! We'll bump up the priority for this feature.

@tyomhak Hi, just wondering if there's any update here, or if there's some other way to accomplish what I want?

@richardrl if you create a report with multiple run sets in a section, those run sets will operate as "OR". Here's an example from our docs. As you can see, each run set is filtering to a group of runs, which are then displayed above together on the charts.

I'd like to note that we do have this ticket in the backlog, and would like to improve the filter UI to directly support that.

@richardrl if you create a report with multiple run sets in a section, those run sets will operate as "OR". Here's an example from our docs. As you can see, each run set is filtering to a group of runs, which are then displayed above together on the charts.

Hi, the problem is I (and I imagine many others) need to do this dozens of time per day, manipulating the report to do it is too time consuming.

I'd like to note that we do have this ticket in the backlog, and would like to improve the filter UI to directly support that.

Is there any update here? Still hurting for this feature. Very hard to compare new runs with key old runs unless I do a bunch of unnecessary tagging. I should be able to see how the "currently running" runs compare with a few select old runs, without generating a bunch of unnecesssary tags.

@anmolmann Any update? Myself and many other researchers I know are hurting for this future.

Would like to echo how useful this would be!

Would also really love such a feature, eg. for setting baselines that have entirely different hyperparameters.
One idea would be to have multiple filter profiles, and only filter out the runs that do not match any of the filter profiles.

Thanks for the context @alinafdima! I've made an internal note for this!

Not sure if this is a new feature, but using the 'in' and 'not in' operators seems to achieve this purpose now.


Don't assume that issuer is a URL

There is an assumption that Issuer is a URL, and no way provided to override.
~The OpenID Connect spec does not require that Issuer must be a URL.~
Above was an erroneous assumption on my part. OpenID Connect Discovery 1.0 incorporating errata set 1 says that a URI with https: scheme is required.
IBM AppID implementation seems out of conformance with the specification.
I can override issuer matching with:
OpenIDConnect.validate_discovery_issuer = false

But there is no way provided to override requirement that issuer is a URL.
From response.rb:
          uri_attributes = {
            required: [
              :issuer,
              :authorization_endpoint,
              :jwks_uri
            ],

          validates *uri_attributes.values.flatten, url: true, allow_nil: true

Workaround is to fork and move :issuer from uri_attributes to attr_required. Or do some nasty monkey-patching.
There should be an easy way to override this validation.
An example of a use case where issuer is not a URL is IBM Cloud (ex-Bluemix) App ID.
Request:
Note I substituted  for real tenant uuid below.
https://appid-oauth.ng.bluemix.net/oauth/v3/<tenant uuid>/.well-known/openid-configuration

{
  "issuer": "appid-oauth.ng.bluemix.net",
  "authorization_endpoint": "https://appid-oauth.ng.bluemix.net/oauth/v3/<tenant uuid>/authorization",
  "token_endpoint": "https://appid-oauth.ng.bluemix.net/oauth/v3/<tenant uuid>/token",
  "jwks_uri": "https://appid-oauth.ng.bluemix.net/oauth/v3/<tenant uuid>/publickeys",
  "subject_types_supported": [
    "public"
  ],
  "id_token_signing_alg_values_supported": [
    "RS256"
  ],
  "userinfo_endpoint": "https://appid-oauth.ng.bluemix.net/oauth/v3/<tenant uuid>/userinfo",
  "scopes_supported": [
    "openid"
  ],
  "response_types_supported": [
    "code"
  ],
  "claims_supported": [
    "iss",
    "aud",
    "exp",
    "tenant",
    "iat",
    "sub",
    "nonce",
    "amr",
    "oauth_client"
  ],
  "grant_types_supported": [
    "authorization_code",
    "password",
    "refresh_token",
    "client_credentials",
    "urn:ietf:params:oauth:grant-type:jwt-bearer"
  ],
  "profiles_endpoint": "https://appid-profiles.ng.bluemix.net",
  "service_documentation": "https://console.bluemix.net/docs/services/appid/index.html"
}

As well, it would be useful to be able to set the expected issuer for cases where it is not a URL, so that we CAN validate the issuer is as expected.
The problem is ~I need to include the tenant ID as part of the configuration endpoint. But~ the returned issuer is:

not a URL
~- and doesn't include the tenant ID~

Which is still a valid response.
I don't mind writing a PR, but would like some guidance as to how you would prefer to approach this.

Issuer MUST be an https url.

FWIW, IBM fixed the nonconformance in AppID API v4.
https://www.ibm.com/blogs/bluemix/2019/04/ibm-cloud-app-id-v4-runtime-apis-update/
The issuer is now a full URL with a protocol and points to your specific App ID instance in the “cloud.ibm.com” domain.
The audience field is now formatted as an array.
Tokens now have a version. This can be found in the header of the tokens in the ver field.
The gender field has been removed from the identity token. If the field is needed for your apps, you can add it to your token through custom claims mapping.
The OAuth client field has been removed from the access token.


multi-target projects

currently tye run fails when trying to run projects that are multi targeted
please consider supporting multi targeted projects or providing an option
at command line to run a specified target

@jkotalik sorry to ping you again on that one but may be this PR would help on that ?
Is there anything left to add to it ?
https://github.com/dotnet/tye/pull/567

@tebeco thanks for pointing this out; I didn't see this.  Q: would tye default to the most current target for a project if no TargetFramework was specified ?

I can tell for the "order of preference"
All I can say is that as a workaround you can specify explicitly the TargetFramework as a buildProperty in the YAML
that will somehow "bypass" the current issue.
The PR would just enable developper to do tye run --framework netcoreappFOO.BAR


scrollbar removed for mobile view

What is the change?
I have successfully removed scrollbar for mobile view i have attached screenshots of before and after
Related issue?
closes: #107
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] I have followed the Code of Conduct.
[x] I have  checked there aren't other open Pull Requests for the same update/change.
[x] I have you tested the code before submission.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm JWOC contributor.

Screenshots (if any)
before

after

@Chaitalishetty please check it

I have successfully removed scrollbar for mobile view . plz check this zip file code
HackClubRAIT-Website.github.io-remove_scrollbar.zip

BEFORE screenshot

after screenshot


Modify sound API to allow full control of sound generation

Actually, Lomse sound API only provides data about midi channel and program. But MusicXML files include a lot of sound and midi information oriented to have full control of sound generation, and Lomse MusicXML importer now deals with all this information.
The Lomse sound API should be modified to add, at least, the midi port and the midi bank. But as this will break backwards compatibility, and so it is an opportunity for a full review of the sound API.

Closed. Time ago I used issues to track my TODO list. This is no longer the case and so I prefer to remove old issues I created.


HTTP sampler corrupts filename with Chinese characters when upload file with http sampler

Nick.Yin (Bug 61669):
precondition：
use multipart/form-data for Post and Browser-compatible headers is chosen.
step:
1.add a HTTP sample to threadgroup
2.choose a file with Chinese character in file upload area in sampler
3. excute the test
excepted result：
the file whose file name is Chinese character is displayed in view result tree
actual:
the file name displays like ?????
by the way: it was working fine with Jmeter 3.1
Created attachment filename1.PNG: filename with chinese charactor

Severity: major
OS: All
Duplicates:

https://github.com/apache/jmeter/issues/4699
https://github.com/apache/jmeter/issues/5144
https://github.com/apache/jmeter/issues/5529

The syntax is to send the filename twice: https://greenbytes.de/tech/webdav/rfc5987.html#examples
We probably need implementing it

The syntax is to send the filename twice: https://greenbytes.de/tech/webdav/rfc5987.html#examples
We probably need implementing it

Can you be more precise? How to send the filename twice? How to do that using BeanShell PreProcessor instead of HTTP Request Files Upload?
Maybe a sample would help

Let me explain:

As seen in https://github.com/apache/jmeter/issues/4546#issuecomment-1256785384, JMeter uses Content-Disposition header
However, historically, Content-Disposition supported ASCII characters only, so RFC 6266 and RFC 5987 suggested a way out to pass unicode

So I would suggest following the examples in RFC 6226: https://datatracker.ietf.org/doc/html/rfc6266#section-5,
 Content-Disposition: attachment;
                      filename="EURO rates";
                      filename*=utf-8''%e2%82%ac%20rates

Then try adjusting JMeter code in https://github.com/apache/jmeter/blob/1e93c67a59673fe4d71a3a710c80bb485965b355/src/protocol/http/src/test/java/org/apache/jmeter/protocol/http/proxy/TestHttpRequestHdr.java#L614 so it generates the same code as in RFC 6226 example.
Do you think you could create such a pull request?

For reference, HttpClient 4.x does not support unicode filenames, and https://issues.apache.org/jira/browse/HTTPCLIENT-293 was integrated to 5.x only.

I've prepared a fix in https://github.com/apache/jmeter/pull/5987.
It would be great if someone could test it with their systems.


Update 7.11.md

One second, WebUI is currently checking if there is another ticket number that needs to be added.


Add in new logo and css styles

Updated the styles (included the new logo). Can revise/change as necessary.

Hey Darcy, thanks for getting this together. You did a ton here! This is rad.
A few issues/bugs would love to clear up before we merge.
Happy to post them here, but may be easier to chat in the architect Slack.
Let me know what you'd prefer, will share wherever!


[Pager] Pager Cannot slide

Description
I tried to use the official infinite sliding example, but I found that when my count is set to 50 or 100, I can slide normally, but if it is set to Int.MAX_VALUE, I can only slide once, and the random sliding will be automatically bounced back to the previous page.
And when I use the CoroutineScope process to turn the page, there is no problem.
Steps to reproduce
Set cout to Int.MAX_VALUE
Expected behavior
I can scroll all the way to the end
Additional context
Column(Modifier.fillMaxSize()) {
           val pageCount = 10
           val startIndex = Int.MAX_VALUE / 2
           val pagerState = rememberPagerState(initialPage = startIndex)
           HorizontalPager(
               count = Int.MAX_VALUE,
               state = pagerState,
               contentPadding = PaddingValues(horizontal = 32.dp),
               itemSpacing = 4.dp,
               modifier = Modifier
                   .weight(1f)
                   .fillMaxWidth()
           ) { index ->
               val page = (index - startIndex).floorMod(pageCount)
               PagerSampleItem(
                   page = page,
                   modifier = Modifier
                       .fillMaxWidth()
                       .aspectRatio(1f)
               )
           }

           ActionsRow(
               pagerState = pagerState,
               modifier = Modifier.align(Alignment.CenterHorizontally)
           )
       }

This problem exists in versions greater than 0.20.3

Seems similar to https://github.com/google/accompanist/issues/945. please try to use some smaller than Int.MAX_VALUE but still large value

Should be fixed by https://github.com/google/accompanist/pull/1058


point to main branch

looks like the toolkit repo switched from master to main so this just updates the README to reflect that

Hey @klauern, thank you for catching that! Apologies for the delay in addressing this, but merging this change in shortly :)


[C++] Regression in PlainBooleanDecoder::DecodeArrow

Describe the bug, including details regarding any error messages, version, and platform.
While looking through another PR, I noticed that we recently introduced a bug in PlainBooleanDecoder::DecodeArrow.
Apparently the tests are not thorough enough to detect the issue.
A possible fix is the following patch:
diff --git a/cpp/src/parquet/encoding.cc b/cpp/src/parquet/encoding.cc
index 6e93b49339..a6e60aa012 100644
--- a/cpp/src/parquet/encoding.cc
+++ b/cpp/src/parquet/encoding.cc
@@ -1208,7 +1208,7 @@ int PlainBooleanDecoder::DecodeArrow(
     BitBlockCounter bit_counter(valid_bits, valid_bits_offset, num_values);
     int64_t value_position = 0;
     int64_t valid_bits_offset_position = valid_bits_offset;
-    int64_t previous_value_offset = 0;
+    int64_t previous_value_offset = total_num_values_ - num_values_;
     while (value_position < num_values) {
       auto block = bit_counter.NextWord();
       if (block.AllSet()) {
@@ -1224,8 +1224,7 @@ int PlainBooleanDecoder::DecodeArrow(
       } else {
         for (int64_t i = 0; i < block.length; ++i) {
           if (bit_util::GetBit(valid_bits, valid_bits_offset_position + i)) {
-            bool value = bit_util::GetBit(
-                data_, total_num_values_ - num_values_ + previous_value_offset);
+            bool value = bit_util::GetBit(data_, previous_value_offset);
             builder->UnsafeAppend(value);
             previous_value_offset += 1;
           } else {

Component(s)
C++, Parquet

We should either fix this quickly for 16.0.0, or revert https://github.com/apache/arrow/pull/40876 . @mapleFU @raulcd

I'd like quickly fixing this

take

I've re-produce the bug using tests, will send a fixing with testing tonight

Issue resolved by pull request 41037
https://github.com/apache/arrow/pull/41037


taffybar is covered by other applications

For example when I opened chromium the taffybar is now covered and is not able to be seen. My xmonad config is here. Is there some mis-configuration in my xmonad.hs?

I think I may have find the problem, it seems like when I do `cabal install taffybar --gloabl" it will automatically install xmonad-0.13 which is breaking this cover thing.

Hooray, I solved it myself.
The solution is I ended up manually set the xmonad and xmonad-contrib dependencies to 0.12.

This is actually a problem now. Because the Debian just updated the xmonad version to 0.13, now the taffybar is suffering from this issue unless we can solve it. I can take a stab but I know very little about haskell. Anyone has any ideas?
Meanwhile I propose we have branches and releases for taffybar. As well as we can make new release and update more features we definitely need to test it on different environment (OS, xmonad, etc) and make it more robust. Do you guys agree?

Glancing over your xmonad.hs it looks like you have only added the manageDocks hook, and did not setup manageDocks as described in the documentation of manageDocks. You can wrap your config with docks and then apply  avoidStruts to your layout.

Done, thank you for pointing out it. We can close this for now.
I also updated the answer so someone else can benefit.

Thx, I think it is solved for now.


Update contribution

Pull Request Test Coverage Report for Build 50

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.54%

Totals

Change from base Build 45:
0.0%

Covered Lines:
1504

Relevant Lines:
1643

💛  - Coveralls


'Add an app' functionality is not coming in the web version of the meeting

'Add an app' functionality is not coming in the web version of the meeting. Because of this, we are not able to test the functionality of our teams app. Is it by design?

@pooran - We are looking into this I will get back to you soon.


Aling to , support

array('FirstElem', 123, true);
array('SecondElem', 2, false);

align to:
array('FirstElem',  123, true);
array('SecondElem', 2,   false);

Adding
        "," : {
            "spaceAfter" : 1,
            "spaceBefore" : 0
        }

to your alignment.chars settings can allow you to achieve the following with the example code:
array('FirstElem' , 123, true);
array('SecondElem', 2  , false);

Not the same, but maybe a bit better than nothing?

I've added Align whitespace command.
Included in v0.3.0.


Resetting playTime

The following code resets playTime to 0 when the Format isn't 2.
https://github.com/nfroidure/midifile/blob/11fd971355e3faa6a912b8546cd0baafba2b1899/src/MIDIFile.js#L90-L91
(That's good, because all tracks should play simultaneously if the Format isn't 2. Edit: The simultaneity of tracks isn't handled by resetting playTime to 0 (and this code isn't reached by Format 1 anyway, although it might get reached by corrupt Format 0 files that have several tracks in spite of Format 0). The simultaneity of tracks is handled by the smallestDelta loop below.)
However, the code comment says the opposite: "reset playtime if format is 2". That comment should be changed, I guess. Edit: I don't know whether the users of Format 2 want to restart the playTime for each track. The comment implies that they do, and in that case the code should be changed to match the comment, although maybe the users have gotten used to the current code behavior. The current code probably has no effect, it sets playTime to 0 only when it's already 0, if I'm not mistaken.
To make the code clearer in that regard, it should be replaced by something like
if (format == 2) {
  playTime = 0;
}

(What seems a bigger problem is that that code is currently inside a if (1 !== format || 1 === this.tracks.length) condition, so it would not execute when it is most important: when Format is 1 and there are multiple tracks. Maybe the current version hasn't been tested on such files, or am I missing something? Edit: Format 1 with multiple tracks is handled by the smallestDelta loop.)
While we're at it, can we rename playTime (and event.playTime) to something like time (and event.time)? "Play" is an appropriate word for Note On events, but most other events don't "play" in the normal sense. It's even confusing for Note Off events, because "playTime" seems like the duration for which the note has been playing.
Edit: TL;DR: The plan was probably to reset playTime to 0 at the start of each track of Format 2 files. I haven't checked whether that's common. In any case, this didn't happen due to a bug, and maybe the users have gotten used to that.

Thanks for discovering this! Yeah, the original formula is really bad style. That is not a notation humans should have to parse.
With event.time, it could be confused with wall clock time of the event happening or being scheduled for. How about position, trackPosition or something like that?

Thanks for helping here, feel free to PR a fix, I'll merge asap.


Provide a way to access secondary index name in lambda function

Environment information
System:
  OS: macOS 15.0
  CPU: (8) arm64 Apple M3
  Memory: 114.58 MB / 24.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 22.8.0 - ~/.nvm/versions/node/v22.8.0/bin/node
  Yarn: undefined - undefined
  npm: 10.8.2 - ~/.nvm/versions/node/v22.8.0/bin/npm
  pnpm: 9.11.0 - ~/.nvm/versions/node/v22.8.0/bin/pnpm
NPM Packages:
  @aws-amplify/auth-construct: 1.3.1
  @aws-amplify/backend: 1.3.0
  @aws-amplify/backend-auth: 1.2.0
  @aws-amplify/backend-cli: 1.2.8
  @aws-amplify/backend-data: 1.1.4
  @aws-amplify/backend-deployer: 1.1.4
  @aws-amplify/backend-function: 1.5.0
  @aws-amplify/backend-output-schemas: 1.2.0
  @aws-amplify/backend-output-storage: 1.1.2
  @aws-amplify/backend-secret: 1.1.3
  @aws-amplify/backend-storage: 1.2.0
  @aws-amplify/cli-core: 1.1.3
  @aws-amplify/client-config: 1.3.2
  @aws-amplify/deployed-backend-client: 1.4.1
  @aws-amplify/form-generator: 1.0.3
  @aws-amplify/model-generator: 1.0.8
  @aws-amplify/platform-core: 1.1.0
  @aws-amplify/plugin-types: 1.3.0
  @aws-amplify/sandbox: 1.2.2
  @aws-amplify/schema-generator: 1.2.4
  aws-amplify: 6.6.3
  aws-cdk: 2.160.0
  aws-cdk-lib: 2.158.0
  typescript: 5.6.2
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables
npm notice
npm notice New patch version of npm available! 10.8.2 -> 10.8.3
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.8.3
npm notice To update run: npm install -g<EMAIL_ADDRESS>npm notice

Describe the feature
Get the secondary index name programmatically.
Use case
To query dynamodb items in lambda function.
This is my model definition:
import { a } from "@aws-amplify/backend";

export default a
  .model({
    id: a.id().required(),
    key: a.ref("ConfigurationKey").required(),
    booleanValue: a.boolean(),
    stringValue: a.string(),
  })
  .secondaryIndexes((index) => [index("key")]);

Secondary index generated in dynamodb:

Hey,👋 thanks for raising this! this should be possible with our Data client but sending this over to our API repository for better assistance.


QuizzEngine - Create an API rest service for loading quizzies

First commit to implement REST API: https://github.com/ProfesorDiegoRosado/quizzies/commit/a770255a740f2b371011eca2db8382c59d94a0a4


Documentation changes related to context code snippets

The way the documentation demonstrates context usage does not work for me (panics on expect).
Its unclear if this is because im running a BrowserRouter in between the context provider and consumers. Either way, adding the Rc makes it work as expected.
This change just concerns the 0.20 version of the docs.

Actually not sure if this is a bug that should be addressed later on or not.

This was a bug... in the code snippet. The types need to be same. It Rc'd later on but not at call site


How to get key to use as URL query parameter inside another cell?

Hi, Thanks for a great component.
Is it possible to extract the key to use in another cell? The cell contents have a link but the link's URL needs the id as parameter.  In this example, is it possible for the colFormatter method to see the subjectId and pass it back in the link?
<BootstrapTable data={this.props.subjectList}  selectRow={{mode: 'checkbox'}} search={ true } multiColumnSearch={ true } striped hover responsive pagination bordered={ false } options={ { noDataText: 'No matching subjects found' } }>
  <TableHeaderColumn dataField='subjectId' isKey={ true } autoValue={ true } hidden >ID</TableHeaderColumn>
  <TableHeaderColumn dataField='subject' dataSort={ true }>Subj#</TableHeaderColumn>
  <TableHeaderColumn dataField='subjectTitle' dataFormat = { this.colFormatter} dataSort={ true }>Title</TableHeaderColumn>
</BootstrapTable>

The solution is simple, in the function to format the column data, just pull the id from the row, so :
`colFormatter = (cell, row) => {
return (

        {cell}

    )
  }`

I thought the row parameter to format function was the row's index, but it's actually the whole object.  This opened up a whole new world. Many thanks @AllenFang.

@devjeff, great, let me know if you got any trouble:)