Sync Engine Support

Feature Request: Support for Synchronous Database Sessions
Is your feature request related to a problem?
No, this is a new feature proposal.
Proposed Solution
I'd like to discuss options for integrating support for synchronous database sessions. I'm open to implementing this myself, but I'd appreciate opinions on the best approach. Here are some potential solutions I've considered:

Separate Synchronous Classes

Create new SyncFastCrud class and sync_crud_router instance
Drawback: This would introduce significant code redundancy

Dual Function Approach (Similar to Langchain)

Introduce separate functions for sync and async operations
Example: create for sync and acreate for async
Implementation: Check if the passed db is a Session or AsyncSession

Artificial Session Wrapper

Create a wrapper to convert sync Sessions into AsyncSessions
Use run_in_threadpool (which FastAPI would do internally for the previous two proposals)

I'm open to other ideas and would appreciate any feedback or alternative suggestions.

I like this feature. I need to think a bit about the possibilities before discussing a best approach

Yes, I agree. I think the most "FastAPI" solution would be to keep the entire sync process synchronous as fastapi will automatically run that in the thread pool and will ensure that no extra "magic" is occurring behind the hood.

@igorbenav Any news on this? Thanks!


Add TracedLayer CN Doc

Related to https://github.com/PaddlePaddle/Paddle/pull/21518.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

New Preview:

Preview:


[Feature request] Preload

Pre-loading is a common use case these days because users expect videos to load instantly in scroll views.
API Details:

GLOBAL: preloadByteCount: specify how many bytes at the beginning of the URL to preload.
GLOBAL: queueMax: Max number of URLS in the preload queue, if this number is reached upon insertion, remove previous URLS to make space
DYNAMIC: networkTaskMax: Max number of concurrent network tasks for preloading at any given time. This is necessary to avoid binging client bandwidth and avoid blocking iOS threads.
preload(urls: Array): add the following urls to to the pre-load queue. Make space in queue if queueMax is reached. ONLY preload if:

the current video playing has sufficient play buffer due to good network. If this is not the case, wait until it is to preload.
the networkTaskMax has not been reached.

stopPreload(urls: Array): remove urls from pre-load queue and stop associated data tasks immediately. Extremely useful if user back out of page for example.

Let me know if you need help implementing these. Should be relatively straightforward.

Place try this:
playerManager.isEnableMediaCache = true

@FoksWang I have, this will cache but not preload.

I understand what you mean. Indeed, this library has no preload features.

@FoksWang any intention of adding it? I could help.

@Ludotrico preload is a good idea, especially for the short videos, I can add this feature, and I will mark it to remind myself. However, in terms of time, I am not sure when I can start to do it, because this year is too busy for me.

@FoksWang I must ask you, why not make your library open source so that others can improve it when you don't have time?
Also, if you do start working on this preload feature, please reach out to me or to a developer that understands the use case well to specify requirements. For example, keeping track of URL tasks and cancelling preloads is extremely important in a tik-tok like video application where videos are in a scroll view.

@Ludotrico First of all, thank you very much for you want to add more features to this library. I created this library to verify a commercial project in the beginning. The commercial code provides CocoaPods support and guarantees that the source code still belongs to the company (private), which is why it is not open source.
I am working on another media project right now this year, so I will just do some bug fixes. New features will not be added until my media project has reached a stage.
Unless fatal issues, it is not the first priority, please understand.


🐛 useNamingConvention matcher does not accept $

Environment information
Biome 1.9.4

What happened?
I'm trying to implement a matcher in useNamingConvention to align to an object member that begins with a dollar sign, using the regex \$(.+), however it's not accepting the escaped $.
Can support for this be included in a future build please? We've got a number of api calls that use this format in our project and, at the moment, they're littered with biome-ignores.
Expected result
\$(.+) is an acceptable matcher
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

This doesn't seem a bug but a feature request?

@Conaclos - oh for goodness' sake, that was it. Thank you!


[DOC-3153] Add currentStatus strategy node expressions

Add currentStatus strategy node expressions
For reviewers, a preview is available.
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@SudarshanPurohit @brian-f-harness are we good on this? Shall we merge this doc PR?

@SudarshanPurohit @brian-f-harness are we good on this? Shall we merge this doc PR?

@SudarshanPurohit asked about adding currentStatus values. I searched, but wasn't able to find the values. I can add them if they are available, or we can merge this PR now and add them later. What do you prefer?


DRF returning HTML on 404 even with drf-standardized-errors installed

I've noticed that the standardized errors don't work if the route doesn't exist. I've made a minimum reproducible example based on the DRF tutorial, which successfully returns the standardized error if you ask for a primary key that doesn't exist:
➜  curl -H "Accept: application/json" "http://<IP_ADDRESS>:8000/users/-1/"
{"type":"client_error","errors":[{"code":"not_found","detail":"Not found.","attr":null}]}%

but still returns html if you ask for a route that doesn't exist
➜  curl -H "Accept: application/json" "http://<IP_ADDRESS>:8000/user/2/"

<!doctype html>

  <title>Not Found</title>

Not Found

The requested resource was not found on this server.

mre.zip

Thanks for raising the issue @scottgigante and providing a sample project.
The problem is that requests to non-existing routes do not reach drf, they are handled by django. So, the drf-standardized-errors exception handler of is never called.
To return the expected format, you can create a custom handler404 that returns the json you want.
This issue got me thinking: what should drf-standardized-errors do about error responses returned by django without reaching drf? like this one or those that happen in middleware... At the very least, some documentation is in order. Maybe provide default error handlers (400,403,404,500) matching what django provides out of the box?

That would be great! On my end it appears this works -- though I'm not entirely sure as I'm not certain how to trigger core Django exceptions for anything other than 404.
# project/app/exceptions.py
import drf_standardized_errors.handler
from rest_framework.decorators import api_view

@api_view()
def error_handler(request, exception):
    """
    Generic Django error handler.
    """
    return drf_standardized_errors.handler.exception_handler(exception, {'request': request})

and then
# project/project/urls.py
handler400 = "app.exceptions.error_handler"
handler403 = "app.exceptions.error_handler"
handler404 = "app.exceptions.error_handler"
handler500 = "app.exceptions.error_handler"

reusing the exception handler in the error handler seems like a nice idea. For now, there is a small issue: for 500 errors, you might see duplicate errors in sentry and duplicate error logs. That's because the handler sends the got_request_exception and django also does that as well, and sentry relies on that to report errors

Good point. Not a huge issue but a little suboptimal. I'd be interested if you come up with something better!

I went ahead and documented this issue in the readme and linked to this issue.
I'm closing the issue as resolved as the solution is up to the developer. Scott's answer is a good start assuming the project provides a backend API only.


chore(anchorScroll): remove a Jasmine toHaveBeenCalled workaround

The Jasmine fix landed long time ago and we've updated Jasmine since that
happened.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Tests update.
What is the current behavior? (You can also link to an open issue here)
A Jasmine workaround is applied.
What is the new behavior (if this is a feature change)?
It's removed.
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Other information:

LGTM


Add error mitigation setting to requests

Enable use of latest symmetrization aggregation in requests.

superseded by #6077


ionic serve not working after installing ionic 2

recently installed ionic2. after that when i try to run/open (i.e. ionic serve) ionic 1 projects which i have done earlier it is throwing me a message: "WARN: ionic.project has been renamed to ionic.config.json, please rename it. Uh oh! Looks like you're missing a module in your gulpfile: Cannot find module 'gulp' Do you need to run npm install?"
I have tried installing bower,gulp, gulp-util. but nothing worked. has anybody got this issue? can somebody help me?

Could you provide me with the output from ionic info from within your project directory? Thanks, this should help in troubleshooting.

Cordova CLI: 6.2.0
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 1.3.1
Ionic CLI Version: 1.7.16
Ionic App Lib Version: 0.7.3
OS: Windows 7 SP1
Node Version: v0.12.7

Did you manage to solve this problem? I've found no answers yet to this very critical problem, how do the ionic team thinks we can build apps with ionic 2 if we cannot maintain our ionic 1 apps?

ran npm install
then did ionic serve, it helped me

make sure you have v2.1.0 of the CLI
ionic -v

I've managed to solve it, I've also put all my procedure here :[http://stackoverflow.com/questions/37468749/ionic-2-ionic-1-work-on-different-node-versions/39765640#39765640]


Describe pros/cons of 🏰 Zitadel

🏰  Zitadel
Who is behind ZITADEL

"Behind ZITADEL is an aspiring team of talented people who have a combined identity space knowledge of over 50 years. Prior of starting ZITADEL the founders worked for a public sector company, where they built and operated a complex service provider grade identity system. This helped the company to provide IT services to government entities, first responders and citizens alike."

Github: https://github.com/zitadel/zitadel

Has a commercial backing: https://zitadel.com/

Winner of Swisscom Startup Challenge 2021 - https://www.venturelab.swiss/Meet-the-winners-of-the-Swisscom-StartUp-Challenge-2021

Releases often, has many releases. Is at v.2.38.2

5K stars, 292 forks.

77 Contributors (https://github.com/zitadel/zitadel/graphs/contributors) (many contributors with hundreds of commits)

Primarily written in Go

Uses Apache License 2.0

Has a dedicated Open Source page on the commercial website: https://zitadel.com/opensource

🇨🇭 Based in the EU - Switzerland

Has a nice comparision page with KeyCloak: https://zitadel.com/blog/zitadel-vs-keycloak

Has a youtube presence: https://www.youtube.com/@zitadel
Intro talk:
https://youtu.be/7gHJZao0hCM?si=FSkILAWvTINLV5ff
Neutral review:
https://www.youtube.com/watch?v=1T1uxKW06Vs

Has both modern gprc and REST APIs

Built with public government in mind 👍

Developed with future proof feature set (e.g. supports passwordless) 👍

Cloud native architecture 👍

Cloud Native DB - https://www.cockroachlabs.com/customers/zitadel/

Selvservice centered design 👍

Note that Zitadel also fully supports PostgreSQL as of version 2.3.0
https://zitadel.com/blog/postgresql-support
Not that PSQL is necessarily the better choice, but worth keeping in mind :-)

Postgre could be a fast and easy solution for the PoC... But for a later cloud-native production setup CockroachDB is preferable.
I have not researched what impact a change in storage solution will incur.. Or how much time it would take to get used to working with CockroachDB. The two investments should be compared before a technical solution is decided.

Authentik has been chosen as the IDP solution for the PoC


Add summaries to historygenerator.py

Thanks very much for your skin and previous help J.
I see that https://github.com/brewster76/fuzzy-archer/blob/master/bin/user/historygenerator.py file now includes a summary column (see http://meteolavall.no-ip.org/weewx/history.html for a sample page which includes it).
I really like your implementation of the NOAA monthly summaries and am not keen to lose that style of presentation.
I've tried comparing the code between your modified version of brewster76's  historygenerator linked above, and this latest one and whilst a lot of it is identical I'm afraid it's beyond my abilities to correctly add the summary column functionality from one to the other.
Any chance you might review at some point and add to your modified version?
My own summary page is here if you are interested - https://www.loughlinstown.ie/records.html
Many thanks,
Muireadach.

Thanks for using my skin. I didn't know he updated his script with a sum column. I believe merged my updated code base for the historygenerator.py before he did this update but I will look into it.

I have the updated version of historygeneratoy.py working on my server. If it remains stable, I will upload the code within a few days.
http://cw9009.x10host.com/summary.html

Nice work!, thank you so much!

Just installed the new file, works great!, thanks again.

Thanks for letting me know about the update.


Unable to configure "COMMAND" or [ARGS] metavars when using click.Group

When using click.Group you can customize options and arguments metavars but you cannot customize the COMMANDS autogenerated metavar in help output when you add commands to the group.
@click.group(options_metavar='<options>')
def cli():
    """A random set of productivity shortcuts"""
    pass

@cli.command(short_help='Some subcommand')
def commA():
    click.echo('This is subcommand A')

It would output:
Usage: cli <options> COMMAND [ARGS]...

  A random set of productivity shortcuts

Options:
  --help  Show this message and exit.

Commands:
  comma  Some subcommand

Resuling in inconsistencies between "", "COMMAND" and "[ARGS]". I would like to be able to produce:
Usage: cli <options> command [args]...

Can you update the @click.group decorator to the following:
@click.group(options_metavar='<options>', subcommand_metavar='command [args]...')
def cli():

Or am I not understanding your issue?

subcommand_metavar

Yes, that works. I guess those options are not in the docs.

Oh great - I was taking a look at resolving your issue and saw that kwarg


Permission for '/train destination' not working

BkCommonLib version: v1.13.2-v2
TrainCarts version: v1.13.2-v1
Spigot version: CraftBukkit version git-Bukkit-ad6070d

Problem or bug:
When granting the train.command.setdestination permission, players do not get access to the /train destination command. This command is instead granted by giving the train.command.properties permission.
Expected behaviour:
Allowing the /train destination command with the train.command.setdestination permission as described in PermissionDefaults.yml.
Steps to reproduce:
Negate the default permission setup (which gives players train.command.properties and train.command.setdestination), then do allow for train.command.setdestination. This will result in players not having access to the /train destination command.
Only allowing train.command.properties will give players access to this command.

that's weird i just looked at the code myself and it's set to the correct permission
https://github.com/bergerhealer/TrainCarts/blob/master/src/main/java/com/bergerkiller/bukkit/tc/commands/TrainCommands.java#L337

Can confirm. although all the underlying code says it should be fine.

Is this issue still affecting you?


iphone release build is showing wrong icons in actual device. But in emulator it is showing correct icons

iphone release build is showing wrong icons in emulator it is showing correct icons
expected icons is forward_ios_arrow
but wrong icon is getting folder_special
flutter doctor -v results
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-x64, locale en-GB)
• Flutter version 2.5.3 at /Users/manjup/Downloads/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 1811693 (6 weeks ago), 2021-10-15 10:46:35 -0700
• Engine revision d3ea636dc5
• Dart version 2.14.4

[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/manjup/Library/Android/sdk
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"

[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:

contents inside file flutter/bin/cache/material_fonts.stamp is below
flutter_infra_release/flutter/fonts/bd151aa3c2f7231344411a01dba4ef61b3cd56b2/fonts.zip

@Manjunath093
Can you provide a minimal complete reproducible code sample along with icons you are using that are not showing up properly ?

here is a piece of code i am using for icons
Icon(
Icons.arrow_forward_ios,
size: 50,
color: ColorConstants.chathams_blue.withOpacity(0.7),
)

@Manjunath093
Tried above code snippet using latest stable and master on iphone device (OS 14.4.1) in release mode and saw no issues. The icon is rendered properly and as-is.


feat / gateway v2

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

Adds the gateway v2 component, under the gateway/ directory.
Adds GatewayEVMAMM connector, which can be used to connect to Uniswap and Pangolin across Ethereum and Avalanche blockchains - and uses Gateway V2 as the middleware to talk to blockchain protocols.
Modifies the amm_arb strategy to work with the new GatewayEVMAMM connector.
Adds new test cases for GatewayEVMAMM connector, and amm_arb strategy.

Tests performed by the developer:

All unit tests passing
End-to-end tests with Uniswap vs. Binance, and Pangolin vs. Binance passing.

This PR comes from https://github.com/CoinAlpha/hummingbot/pull/185
Snapshot proposal: https://snapshot.org/#/hbot-prp.eth/proposal/0xf4120a45f2d563b010e5392e3a56171c700867b568a4e4850253e8bb75bc2833

Hi, please resolve conflicts in this PR. Thanks!

Done - merge conflicts are resolved.

Seems like there is another one @martinkou. Sorry for the issue, but can you please resolve the conflicts again?


Support full set of selectable input types

JSDOM does not support the full list of selectable input types (according to WHATWG) and throws an exception if an unsupported element is selected. Please update the _allowSelection condition to include the full list of supported input types.
List of selectable input types according to WHATWG:

Text (type=text)
Search (type=search)
URL (type=url)
Telephone (type=tel)
E-mail (type=password)
Password (type=password)
Date (type=date)
Month (type=month)
Week (type=week)
Time (type=time)
Local Date and Time (type=datetime-local)
Number (type=number)
Color (type=color)
File Upload (type=file)

/source https://html.spec.whatwg.org/multipage/forms.html#the-input-element
Besides the condition update I suggest to fail select calls on unsupported input types silently as propose by WHATWG:

In the case of input elements, if the control has no text field, then the method must do nothing.

/source https://html.spec.whatwg.org/multipage/forms.html#dom-textarea/input-select
Many thanks in advance! :bow:

Pull request welcome! Should not be too difficult to change the list coded into the source.

Yeah, should be easy. I will try to give it a shot in the next days, but can't promise anything.

I'll hop on this if you don't have time @orlandohohmeier, I helped with the original implementation.

For input elements, calling these methods while they don't apply, and getting or setting these attributes while they don't apply, must throw an InvalidStateError exception.

Can't avoid the exception, unfortunately. I'm working on implementing this now.

Also rewrote the tests again for this: https://github.com/w3c/web-platform-tests/pull/2591. I thought I added these in my PR a while ago @domenic?

Hmm I don't see them in https://github.com/tmpvar/jsdom/tree/master/test/web-platform-tests/to-upstream/html/semantics/forms, so not sure.

This can be closed I think.


adding ash support to foundryup install.sh for easier install/use on busybox/alpine

Motivation

This adds support for ash (the Almquist shell, as in, on Alpine linux/busybox) in the installer. This allows you to use foundry in a busybox without installing another shell.
Solution

I add it as a shell option in the installer.
I tested this with this Dockerfile, and everything went fine :) have to do some futzing later on with paths for the forge binary, but it installs and runs.
FROM alpine

RUN apk add --no-cache bash curl jq git

# RUN curl -L https://foundry.paradigm.xyz | bash
# replace the previous line with the following to test my patched version :P
COPY ./integration-test/install.sh /install.sh
RUN chmod +x /install.sh
RUN /install.sh
ENV PATH="/root/.foundry/bin:${PATH}"
RUN foundryup

when your using an interactive session, POSIX says you will go to profile anyways if bash is unavailable. This is how Ubuntu works, as their default shell is dash
Here is a diagram of process invocation for reference:

im gonna tape this to my bedroom wall above my bed so i can stare at it
before i fall asleep
On Fri, Oct 7, 2022 at 12:14 AM s bacha @.***> wrote:

when your using an interactive session, POSIX says you will go to profile
anyways if bash is unavailable. This is how Ubuntu works, as their default
shell is dash
Here is a diagram of process invocation for reference:
https://raw.githubusercontent.com/sambacha/dotfiles2/master/.github/interactive_vs_noninteractive.png
—
Reply to this email directly, view it on GitHub
https://github.com/foundry-rs/foundry/pull/3418#issuecomment-1271118319,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDPMJCBVYFCEK57LFM7KMLWB6WSZANCNFSM6AAAAAAQZNQYIY
.
You are receiving this because you authored the thread.Message ID:
@.***>


how to use onRead for securing file access?

  collection: Avatars,
  name: 'Avatars',
  onRead: (fileId, file, request, response) => {
    response.writeHead(403);
    return false;
  },

Given the code above I would expect that this would disallow all file access to avatars instead I received an error in the console because of the response.writeHead method.
(STDERR) Error: Can't set headers after they are sent.

Hi @sbalbalosa, this is exactly what it is supposed to do, are you sure you don't have any error in the console other than the one you mentionned ?
I tried to do the same thing and it works fine on the https://github.com/jalik/ufs-example test project.

@jalik recently I noticed that the package I installed was outdated, I updated the package and now it works fine. I think it is because of outdated dependency for jalik/jalik-ufs-gridfs this is the adapter that I'm using.

@sbalbalosa good news that you solved it, also don't forget to install dependencies like gm and everything should works like a charm.


Support Sketch 45 plugin update system

What is Sketch 45 plugin update system ?

@thierryc see http://developer.sketchapp.com/introduction/updating-plugins/

Thank you !


problem with pad

hello，When I am using the pad, the horizontal screen display, the bubble is not close to the screen

If I understand correctly, the problem you're facing is that the bubble is behaving incorrectly in landscape mode, is that correct?

hello, yes, when i click this button while in portrait and then i rotate to landscape, the bubble doesn't get close to the edge of the screen

It's definitely a bug. I'm surprised I didn't notice it earlier! 😅
Thank you for bringing it to our attention!

hi @Honglixi99 , 👋
This bug has been fixed in v0.5.3.
However, it still has some minor UI issues that I'll fix later.
Thank you.

can I lock it to portrait

@rajesht4101999 nope unfortunately, that's how android work

but when I change to landscape expanded bubble ui becomes half cut @TorryDo

@rajesht4101999 can you share a video?

https://drive.google.com/file/d/1IyBRsFoOcwu9I-C8F67DKSqwc779P_zu/view?usp=sharing
when I trigger to start bubble service in landscape mode


[Tests] Add teardown to Backend test

Let's add teardown to Backend test to avoid remaining test mockup
backend.
ONE-vscode-DCO-1.0-Signed-off-by: Yongseop Kim<EMAIL_ADDRESS>

PTAL @Samsung/one-vscode


JsonSerializer private in EntitySerializer

Hi,
I would like to add some custom JsonConverters to the jsonSerializer, because my entities have types that cannot be converted to json with the default jsonconverters. Is it possible to make the JsonSerializer public in EntitySerializer? Or is there another way to add converters to the serializer?

Actually there was a point when I considered that. In the end I didn't need that though and always use the attribute
public class MyEntity
{
    [JsonConverter(typeof(MyConverter))]
    public CustomValue MyProperty { get; set; }
}

Is this something you can do in your code?

Ah, and I think that you place that attribute on a class too


HDDS-6955. [Ozone-streaming] Add explicit stream flag in ozone shell

What changes were proposed in this pull request?
Cluster-level replication for EC was introduced in HDDS-6294.
When client side and bucket level replication config are both missing,
client will pass null as replication config to let OM decide.
The first problem is NPE in createStreamKey() if replication config is null.
The second problem is, since streaming is not compatible for writing EC keys,
we cannot determine when to use streaming just by key size and client side replication config.
In this PR, I have added a explicit --stream flag in ozone shell key handler.
In stream mode, replication config will fallback to client side defaults when missing.
And it will be checked to ensure not using streaming to write an EC key.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6955
How was this patch tested?
Manual test
ozone sh --verbose key put --stream /vol1/bucket1/key1 data

@captainzmc @szetszwo PTAL, thanks.

@szetszwo Thanks for the improvement.

Thanks @kaijchen working on this.  Since HDDS-4454 has been rebase, can you rebase the PR?

Thanks @kaijchen working on this. Since HDDS-4454 has been rebase, can you rebase the PR?

Sure. I have rebased this PR to HDDS-4454.

Thanks @szetszwo and @captainzmc for the review.


fix for freeCamera pointer lock and fullscreen

http://www.html5gamedevs.com/topic/25108-problems-with-unlocking-pointer/
http://babylonjs-playground.azurewebsites.net/#15ECGN#13

You can easily see the current issue with the freeCamera controls by going to the playground and then pressing the fullscreen button.  The freeCamera controls won't work in fullscreen.

Top answer your second question - yes, that would be the best way to go. Try creating a new branch per pull request, otherwise every commit will be shown here.
Regarding the code - we'll review it and let you know. Without testing - it looks fine.

@abow: I have a pending question in your code

What's the question?

Check the code. I started a review

Where do I find the review?  I'm not seeing it here:
https://github.com/BabylonJS/Babylon.js/pull/1352/files

I'm not sure why I can't view that question on my page.
Anyway, we need to use mousemove, because the following doesn't work with pointermove:
var offsetX = evt.movementX || evt.mozMovementX || evt.webkitMovementX || evt.msMovementX || 0;
var offsetY = evt.movementY || evt.mozMovementY || evt.webkitMovementY || evt.msMovementY || 0;

Ok makes completely sense.
I will merge your PR but you need first to fix the conflict :)

(and please credit yourself here:https://github.com/BabylonJS/Babylon.js/blob/master/dist/preview release/what's new.md)

Something went wrong with your commit. 39 files updated and you are reversing a lot of recent changes

I'm having trouble with git.  :(
I didn't mean to commit those js files.

Hi, conflicts seems to be gone, but you are still commiting a lot of files (39) for now reason. Can you try fixing that?

I think I finally got it.

I have one more commit I'd like to make later today that would fix the issue with going from pointer locked to pointer unlocked.  Right now, you have to click to stop the camera from moving on mouse move.


chore: add bootstrap address for diszell2008

Description
Briefly describe the changes introduced by this pull request.
Related issue(s)
If this Pull Request is related to an issue, mention it here.

Fixes #(issue number)

@diszell2008
The address is invalid. Please provide the correct address

@diszell2008
Please enable relay and auto nat services.
Please refer to this document for more help on running Bootstrap node: https://github.com/pactus-project/pactus/blob/main/docs/bootstrap.md

Enabled sir!

@diszell2008
Merged with thanks


[FEATURE] Prometheus metrics for backups

The feature request for Prometheus Metrics support already mentions metrics for backups. It would also be great if there were metrics for the size of the backups too. So something like longhorn_backup_actual_size.

the backups are based on snapshots if I understand the documentation correctly. So it would be cool to have both snapshot and backup metrics. The snapshots are incremental so here the size is also a matter. Some use cases:
If the snapshot size(incremental) is for x-time 0 bytes, than you could say this volume isn't used. For some systems like databases and so on, which are in use, this could signal a bug or misbehavior in the software.
If we know the successful backup timestamp plus the recurring time window we could calculate n-times of recurring time window are allowed to fail until alert.

@jenting Please help describe the test cases here for QA reference. Thank you!


Please explain how to work with BGFX_STATE_BLEND_INDEPENDENT.

Hi!
Please explain how to work with BGFX_STATE_BLEND_INDEPENDENT.
What are the opportunities?
Need write to GBuffer with the ability to blend values.
Simple example:
gl_FragData[0] = encodeRGBE8(baseColor);
gl_FragData[1] = vec4(normal * 0.5 + 0.5, 0);

How to disable writing only to gl_FragData[1]?

How to configure blending? Need something like:
new gbuffer value = lerp( old gbuffer value, new value, 0.5);

Is this possible to set blending factor from fragment shader? Or it is possible only from C++?

Judging by the code, there are many possibilities. Although there is no example on the Internet.
In our case, it was necessary to set the recording mask for each element in the buffer. 5x4 bit values = 20 values (5 RGBA targets). This can be done by changing the behavior of Bgfx code.


no iPad for MIRA

How required is MIRA?
As I understand MIRA is an app that just allows you to have a configurable midi controller interface, but this is the first time I'm seeing it.
I want to try this project but do not own an iDevice.  Thank you.

Mira is not required, it is just for ease of controlling the max patch as you mentioned.


Bug report: VLC CLI broken

Description of issue
VLC works for me correctly if I use it from graphic interface, but it doesn't work as command-line app:
$ vlc -v
VLC media player 2.2.2 Weatherwax (revision 2.2.2-3-gf8c9253)
[000000010022d698] core libvlc warning: cannot read /opt/homebrew-cask/Caskroom/vlc/2.2.2/VLC.app/Contents/MacOS/lib/vlc/plugins/plugins.dat: No such file or directory
[000000010022d698] core libvlc error: No plugins found! Check your VLC installation.

Output of brew cask install vlc --force --verbose
==> Downloading https://get.videolan.org/vlc/2.2.2/macosx/vlc-2.2.2.dmg
Already downloaded: /Library/Caches/Homebrew/vlc-2.2.2.dmg
==> Verifying checksum for Cask vlc
==> Symlinking App 'VLC.app' to '/Users/honzajavorek/Applications/VLC.app'
==> Symlinking Binary 'VLC' to '/usr/local/bin/vlc'
🍺  vlc staged at '/opt/homebrew-cask/Caskroom/vlc/2.2.2' (1459 files, 118M)

Output of brew doctor
Your system is ready to brew.

 If you use the full path instead of linking to it, as they do in their examples, it does indeed work.
This means that linking is the issue, here, and that binary does not work when linked to, instead of being called directly. That should likely be considered a bung on their part. Since we won’t change your shell’s startup files for you, this needs to be removed and handled a different way (will open an issue soon).


Remove requirements.txt

Feature request
Once it's no longer useful
Motivation
No response

This will be addressed in : https://github.com/OpenAdaptAI/OpenAdapt/pull/344


Refactor/rc

:tada: This PR is included in version 14.0.0 :tada:
The release is available on:

14.0.0
GitHub release

Your semantic-release bot :package::rocket:


Update pom.xml

Each module of personium doesn't use library of same version, and used libraries are old.
Therefore, we need to review pom.xml.

Released in v1.4.11


Feature/80 make form functional

What has changed and why?
Updated content
openfisca aotearoa data
style tweaks
How to test this change

[ ] has automated tests
[ ] at least one a reviewer ran the code
[ ] tested in small screen
[ ] i18n
[ ] tested in screen reader/contrast <-- remove this if no UX change

@Br3nda @verbman @JacOng17 I will tidy up duplication in another branch. Ready for review! :D


History & get_next_task refactor

It would be nice if locust could handle dependent tasks through get_next_task. To allow this feature in a extensible way, the following PR contains three complementary changes:

TaskSet._task_history persist each of the tasks that goes through execute_next_task
task_callable is optional for schedule_task
get_next_task receives an extra argument called last_task

Hi!
From what I can tell (I've only skimmed the code though), there's a possible memory issue with the task history list growing huge for long running tests.
Could you give some examples of when this would be needed? Could this be achieved by subclassing TaskSet in the locust test files instead?

Hi! This PR has been untouched for a very long time so I am closing it. Let me know if this was a mistake and feel free to open it again.

@cyberw Hi! It has been a few years since I don't use Locust, so that's why I didn't follow up this PR (sorry). FYI: I ended up using this code in production and I didn't have any issues.
I think that you should keep it closed if no one needs this feature or you think it cannot be improved. As @heyman noted there's a possible memory issue if the task history list is not controlled (somehow).
The kind of tests I needed to perform were quite complex and require to do different flows depending the scenario (different responses and states). Defining a new TaskSet subclass was not an good option since I have to remove a large amount of code from run() and schedule_task()


Sdk 613 parallel tests

Functionality to run integration tests in parallel

PR is substituted by: https://github.com/HorizenOfficial/Sidechains-SDK/pull/688


Fixed required trait constraint not being checked

Description
An optimization on check_if_trait_constraints_are_satisfied_for_type_inner was making missing trait constraints not being detected.
Fixes #6374.
Checklist

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).

[ ] If my change requires substantial documentation changes, I have requested support from the DevRel team

[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

Should we add a test of the repro from the original issue?


Move deployment temp file to temp folder

Create deploy zip in temp folder in case it was occupied by other process and unable to delete after deployment.

Codecov Report

Merging #867 into develop will decrease coverage by 0.02%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop     #867      +/-   ##
=============================================
- Coverage      44.55%   44.52%   -0.03%
  Complexity       765      765
=============================================
  Files            134      135       +1
  Lines           5169     5172       +3
  Branches         744      744
=============================================
  Hits            2303     2303
- Misses          2692     2695       +3
  Partials         174      174

Impacted Files
Coverage Δ
Complexity Δ

.../com/microsoft/azure/maven/webapp/utils/Utils.java
0% <0%> (ø)
0 <0> (?)

...ebapp/handlers/artifact/ArtifactHandlerImplV2.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

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 1262b52...486f789. Read the comment docs.


Update for 1.12.3 release

Description
Urgency

[ ] Blocker
[x ] High
[ ] Medium

Requirements

See the contribution guidelines.
Build content locally and test for formatting/links.

don't merge yet


Post release automated changes for compute releases

Post release automated changes for azure-rest-arm-compute

@xirzec do you know why this PR is asking for your review not us ?


Fix for users.profile.server.controller.js security (#1338)

Fixes #1338 an issue where if req.body._id was not set to the current user it
could potentially log the current user in as another user.

Coverage increased (+0.03%) to 70.647% when pulling 7f36aa1b115b2c466b83e83b0dc1b715696f3804 on masterwok:master into 5da5a6106f11e93fa8fa5adb34e96f62725b6a9d on meanjs:master.

This PR fixes https://github.com/meanjs/mean/issues/1338

@masterwok I can't merge it as is because you need to style your commit message according to our guidelines.  Thanks @masterwok.

@masterwok care to update this PR please?

@lirantal @mleanos  Sorry I sort of suck at using git and can't figure out how to rebase my commit  or update my pull request.

Coverage decreased (-0.6%) to 70.022% when pulling 6adede37c8b2a66c8d48730e4cb721edf695591a on masterwok:master into 5da5a6106f11e93fa8fa5adb34e96f62725b6a9d on meanjs:master.

@lirantal When you merge, if you choose squash, I'm fairly confident you can modify the commit message now as well. (I could be totally wrong though too.)

Coverage increased (+0.03%) to 70.022% when pulling 27de351a911f30a2fb4983544d04dc9a5e765a9c on masterwok:master into c8cbcd326f4ab4253a090fbafc2d0af336471258 on meanjs:master.

Coverage increased (+0.03%) to 70.022% when pulling 27de351a911f30a2fb4983544d04dc9a5e765a9c on masterwok:master into c8cbcd326f4ab4253a090fbafc2d0af336471258 on meanjs:master.

Thank you @masterwok
I've merged this very important security fix, hope to see more contributions :)


research: backend fixes for the "available" profiles endpoint

User Story
As detailed in https://github.com/chef/automate/issues/653, the available profiles endpoint is taking much longer than expected to return results. We have ideas of how to make this a bit better on the front end (workarounds), but let's do some investigation into what we can do on the backend to make it faster. Not like we're doing anything fancy in this endpoint; it shouldn't take this long.
Definition of Done
Available profiles endpoint is faster, or we have reasons why it is the way it is.

medium t shirt size

@phiggins and i discussed this issue last week
he is going to work on adding pagination and search filtering to the endpoint

i am going to open a new issue for the work, since we expanded the scope quite a bit


Changelog Visualization issue

Environment

Python version:  3.6.3
Peering Manager version:  0.99

Steps to Reproduce
Log into the system and make a log of changes
Expected Behavior
scroll  changelogs and they will stay inside the container
Observed Behavior
Changelog list goes beyond the container until you get to the end of the list

th

I confirm that this UI bug can be seen with several browsers running on different OSes. I did find the CSS trick yet so if someone has any clues or even a fix I'll be happy to talk about them.

I'll be happy to help if that's possible

I also noticed this issue, seems to be persistent on all OSes and browsers. CSS related?

So I have done more testing, and it seems that this problem only happens on windows computers, not sure about linux yet, on mac this works fine without issues

I have the issue on macOS with Chrome but not with Safari.

I have the same as @respawner, also an issue in Firefox on both macOS and Linux.


Sort the subs in directory lol

Dunno how, really. alphabetical is easiest ofc but there's an argument for and against popularity or something


Setup an easy way to test the app with keycloak / authentication

This is needed to properly test https://github.com/inmanta/web-console/issues/1724

Hmmm, in the past we had a fake API response in a json file where we updated the userID/JWT-token. This file was in the git-ignore to avoid it landing in the repo. But I don't know in which extent this would be applicable for us.
We didn't yet use Keycloack, so we were doing the auth call in postman or a custom script that would use credentials stored in a .env file.


Fix link errors by inlining non-template functions

Stop link errors with the  make_expected and make_unexcepted_from_current_exception functions by declaring them as inline.

Thanks Isaac!


chapter 6 Cannot destructure property 'invoices' of 'require(...).default' as it is undefined

Hi everybody I'm getting this issue when I running npm run seed :
$ npm run seed

seed
node -r dotenv/config ./scripts/seed.js

D:\NextJS\nextjs-dashboard\scripts\seed.js:2
const { invoices, customers, revenue, users } =
^
TypeError: Cannot destructure property 'invoices' of 'require(...).default' as it is undefined.
at Object. (D:\NextJS\nextjs-dashboard\scripts\seed.js:2:9)
at Module._compile (node:internal/modules/cjs/loader:1376:14)
at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
at Module.load (node:internal/modules/cjs/loader:1207:32)
at Module._load (node:internal/modules/cjs/loader:1023:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:135:12)
at node:internal/main/run_main_module:28:49
Node.js v20.11.1
Can you help me please !

If you have this issue just change this line is the seed.js.
const {
invoices,
customers,
revenue,
users,
} = require('../app/lib/placeholder-data.js');
The ".default" is not necessary.


Add target indicators support

Closes #5.

Force-pushed to:

Fix StyLua issues
Rebase
Squash commits


add support for client ssl with wsse security

Combine Client SSL add options to header and WS security together.

Coverage decreased (-2.8%) to 90.324% when pulling 5faf44c10084eccce300ca1219cca94d1e390564 on wellszhane:master into f7d01dfa0b4c37a8f473e96482cead4c8b2753a7 on vpulim:master.

Please add tests if you can @wellszhane .

+1
Multiple questions for this on Gitter. Same here.


emoji: Automatically convert text to emoji

Convert :) to the simple smile emoji and ;) to the wink emoji. Feel free to suggest more if there are any commonly used ones.

@zulipbot claim

Duplicate of #1768?


GUVNOR-2273 : Replace "package name white list" with Dependency Filter and related UI improvements

Scope is no longer editable in the dependency grid.
You can white list the packages in the listed jar or remove from the white list.
Transient dependencies are also listed.
Packages in the current project are always white listed, but everything else needs to be added.
Also removed some redundant code and split few classes into smaller units.
https://github.com/droolsjbpm/kie-docs/pull/32
https://github.com/droolsjbpm/guvnor/pull/219
https://github.com/droolsjbpm/kie-wb-common/pull/202
https://github.com/droolsjbpm/drools-wb/pull/108
https://github.com/droolsjbpm/jbpm-console-ng/pull/261
https://github.com/droolsjbpm/jbpm-form-modeler/pull/32

Couple of small suggestions; otherwise +1 to merge.

Fixes in.

+1 to merge.

https://github.com/droolsjbpm/guvnor/commit/4c5b75932c0d8c9765f9c7202268435901a937a3

Looks like there is one other PR in the chain which we were waiting for https://github.com/droolsjbpm/kie-wb-common/pull/202. The build of kie-wb-common is currently broken because of that.

...and there's one for drools-wb (IIRC Rikkola was looking into failing
tests on Jenkins for kie-wb-common?)

@manstis @psiroky there are 5 more PRs here and we're waiting for @Rikkola to update https://github.com/droolsjbpm/kie-wb-common/pull/202. @porcelli I guess you merged this too early. So best if I merge all the other ones, I guess and we'll fix the test issue later?

I'm happy with that - I have some work in progress (BPMSPL-252) that needs
some heavy refactoring following these PRs.. and would prefer to get done
sooner that later :-)

my bad, as I maybe did it too early. BUT in the other handm my regular process to merge has been passed, I meant... it build local (mvn clean install), what happens is sorta new to me is that we're doing -T2C - and this is faling.

@porcelli no worries. I will merge the others. This is somewhat of an edge cases with that many PRs involved. However, I did experience the kie-wb-common test failures on my local machine even without -T....so something is up there. We have to investigate that separately. /cc @manstis @psiroky @Rikkola

I'm looking at the test failure issues. I did manage to reproduce them once. Feels like they are random.


Revamp Color Theme and Docs/Features

This PR primarily revamps the use_preferred_color_scheme hook into use_system_theme.
Additionally, this PR features a few other things such as:

Improved docs
Feature name changes
All tests passing!
Fixes such as including the notification module (how did this get excluded?)
Clippy fixes.
And probably some more.

*Has breaking changes.
Closes #35

I have added the fix for window size. It is ready for its final review and then we can merge 🙂


Failed to stop tracking time

Hi there,
thanks for the project. It looks nice.
However I encountered a bug trying to stop tracking:
timetrace create project test
✔ Created project test
timetrace start test
✔ Started tracking time
timetrace stop
❗ Failed to stop tracking: record already exists

timetrace status works fine.
The json looks like this:
{
	"start": "2021-05-26T10:23:37.5685406+02:00",
	"end": null,
	"project": {
		"key": "test"
	},
	"is_billable": false
}

I use timetrace version 0.6.0 on Windows 10.

Can confirm this bug with timetrace v0.6.0 on macOS 11.3.

This bug has been fixed with release 0.6.1. Just download the latest release,timetrace stop should work again.


Allow controllercmd to reuse default config and rotation logic

A controller that needs to slightly tweak the builder (to provide
either extra API serving, health checks, or alter authorization)
must currently clone the StartController() method. Instead, split
config and default rotation into their own methods to make it
it easier to turn the default command into config, then start
default rotation.
@deads2k when I tried to reuse had to copy the rotation stuff
to tweak the builder

/hold
Doesn't do exactly what I need.

/hold cancel
Had a bug, I was calling SetDefaultRotation after the builder, but SetDefaultRotation actually defaults the config when dealing with self-signed certs.  New order is correct.

Minor comments.  The names are actually confusing and I think for a Setting that doesn't set, the doc is important.
lgtm otherwise, self-tag after fixing.


Broaden access to ArrayGenerator attributes in Python

This is for #272.

I think this fixes #272; I think it's enough to have all of these accessors on the ArrayGenerator and not the VirtualArray:
https://github.com/scikit-hep/awkward-1.0/blob/33c73c9c4e854b81b760c286431d516237508dfa/tests/test_0274-access-ArrayGenerator-in-Python.py#L13-L48
There's a strong argument for putting them on the generator, because there are two generator types accessible in Python (a Pythonized ArrayGenerator and an opaque SliceGenerator, which can't easily provide access to its slice because I have Python → C++ slice conversions but not in the other direction; that would be a big project). The with_ methods let you change the Form and length without needing to know which type of generator you have. If it's a Pythonized ArrayGenerator, then you can change the callable, args, and kwargs, too.
On VirtualArray, however, there's a generator property that gives you the generator (regardless of its type) that you can then call with_* on. Having to rebuild a VirtualArray from all of its components to just replace the length might be a chunky bit of code, but it would be much easier to wrap that up in a helper function in Python than to build a full set of with_ functions at this next level up.
Does this work for you? If so, I'll merge it immediately.

Forgot to add @nsmith- to my previous comment.

Yeah since length is a generator property and not a virtual array property, I think it makes sense to have with_length on the generator. There is not so much work to rewrap as the virtual array properties all appear accessible already. I suppose the following is sufficient:
def add_length(array: awkward1.layout.VirtualArray, length: int):
    return awkward1.layout.VirtualArray(
        array.generator.with_length(length),
        array.cache,
        array.cache_key,
        array.identities,
        array.parameters,
    )

I checked out this branch, and ran into another issue that is familiar from awkward0: the form check upon materialization in my parent array is not happy that the expected form has has_length=False for some of the children nodes where the materialized one has has_length=True.

I checked out this branch, and ran into another issue that is familiar from awkward0: the form check upon materialization in my parent array is not happy that the expected form has has_length=False for some of the children nodes where the materialized one has has_length=True.

That, in itself, is a good argument against making the length mutable.

If I'm willing to lie to awkward and give a dummy length while making the original form then it "works" but I'm not sure of the implications here.

The only implication is that if it is materialized before you expect it to be, there would be an error due to the dummy length being the wrong length.
I was thinking that I should weaken the constraint for you: to make has_length be three-valued: true, false, and "I don't care." But, thinking about it, isn't "I don't care" the same as has_length=False? If you have nested VirtualArrays, the outer one can have a different has_length than the inner one.


FAM - FACULDADE DE AMERICANA

Solicito a inclusão a minha faculdade na lista de vocês.
FAM - FACULDADE DE AMERICANA

@dalleth-martinss I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host fam365.com.br.domains.lib.file not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Beta Documentation - Rest endpoint (async) missing

Documentation Issue
async flag is missing in the beta documention for https://payloadcms.com/docs/beta/rest-api/overview
Is:
](handler: (req) =>  {)
Should be:
](handler: async (req) =>  {)

Thanks for the report, updated!


Pages to be Migrated

[x]  DOI Kernel XML Schema (https://www.doi.org/doi_handbook/DOI_Schema_Release_Notes.html)

[x]  DOI Kernel XML Schema Policy (https://www.doi.org/doi_handbook/DOI_Schema_Policy_Notes.html)

[x] DOI Data Dictionary (https://www.doi.org/doi_handbook/schemas/dd/intro.html)

This is now complete. See

DOI Kernel XML Schema: http://localhost:1313/the-identifier/resources/handbook/doi_kernel_xml_schema/
DOI Kernel XML Schema Policy: http://localhost:1313/the-identifier/resources/handbook/doi_kernel_xml_schema_policy_notes/
DOI Data Dictionary: http://localhost:1313/the-identifier/resources/handbook/doi_data_dictionary/

On the Data Dictionary we've just brought over the raw HTML of the dictionary itself, which comprised of about 550 pages. This could perhaps be migrated into a new-look template, or rebuilt in some way, after launch.

I've just migrated the first 2 links Jonathan. The third one - https://github.com/doi-foundation/doi-website/issues/doi.org/doi_schemas/iso3166a2.xsd - is giving me a 404. Can you provide an updated URL?

https://www.doi.org/doi_schemas/iso3166a2.xsd

Great, thanks for that. Now migrated: https://staging.doi.org/doi_schemas/iso3166a2.xsd


Module docstrings

Pep257 discusses documenting functions, classes and other objects exported from a module should be documented in the module docstring. Add a linter that checks this:
# module.py

"""...

Functions:
    foo: ...

Classes:
    FooClass: ...

Constants:
    FOO: ...
"""

FOO = ...

def foo(): ...

class FooClass: ...

Maybe support abbreviation Funcs and Clss?

And also supports Consts abbreviation?

Private constants, functions and classes would be optional, indicated by a leading _

Documenting constants would also be optional since many constants's name could be enough documentation for them


Test buffered reader / Ensure EmptyBuffer does not get content in it

This PR introduces tests for bufferedReader to ensure there are no regressions, additionally, it adds a panic() case where we've attempted to use the downloadBody or a .readFrom() on the bufferedReader when emptyBuffer is the underlying bytes.Buffer.
The panic() is indicating a programming error, additionally it helps ensure the assumption that emptyBuffer will EOF is guaranteed. The only use of the br.buf.ReadFrom is from tests and/or downloadBody.

Ooof, bufferedReader sure is hard to test, isn't it? This PR made me feel bad enough about its interface that I have raised #180 to change it to have a better interface (purely on io abstractions with nothing http-specific) and also a better implementation (delegating more logic to bufio.Reader).  If we do that, we don't have to test for some of the behaviour we're testing for here - there's no longer an emptyBuffer that we might write to, we don't have to implement checks for full buffers, bufio.Reader has a fixed-size buffer so we can't unintentionally grow it, bufio.Reader handles passing errors during fetch out on the Read() interface appropriately, etc.


Holding page

Need a "coming soon" home page for now, until it goes live.

I still want to get to the 'real' home page if I'm logged in though.
So:

Create a home page that sees if logged-in, and if so it goes to the 'real' home page
Otherwise, it just puts up a simple template with 'coming soon' and an admin login button.

Need to not do this in test env, or the tests'll break

Update, don't have a login link there, as it presents the same problem. I can go directly to the login page if required.
If anyone else finds it by guessing, or by looking at the source, let them - it doesn't really matter if they really want to be nosy.

Fixed by PR #247


addAutoIncrement generated wrong sql

Environment
Liquibase Version:3.8.8
Liquibase Integration & Version: run as jar File
Liquibase Extension(s) & Version: NO
Database Vendor & Version:PostgreSQL 11.1
Operating System Type & Version:Windows10
Description
The following change set produced Exception because the generater sql syntax is invalid.
//it s working for db2 but doesn't work for POSTGRESQL
Steps To Reproduce
<addAutoIncrement  tableName="MYTABEL"	columnName="ID" defaultOnNull="true" generationType="ALWAYS" incrementBy="1" startWith="1"/>

Actual Behavior
CREATE SEQUENCE public.MYTABEL_id_seq AS null NO CACHE  NOORDER
liquibase.exception.MigrationFailedException: Migration failed for change set path-ignored::db-changelog-tables-1.0.0::Reza:
Reason: liquibase.exception.DatabaseException: ERROR: syntax error at or near "null"
Expected/Desired Behavior
ALTER TABLE MYTABEL DROP COLUMN ID;
ALTER TABLE MYTABEL
ADD COLUMN ID SERIAL PRIMARY KEY;
COMMENT ON COLUMN  MYTABEL.ID IS 'The entry ID';

Hi @reza-sp Thanks for writing up this issue.  We will add it to our list of issues to prioritize and update.

I stumbled over an issue that is probably tightly related.
<changeSet author="Me" id="switchToPKAutoIncrement">
		<addAutoIncrement tableName="MyTable"
			columnName="id"
			generationType="ALWAYS"
			incrementBy="1"
			startWith="100"/>
</changeSet>

The generated SQL, however, is missing the startWith and generationType information:
CREATE SEQUENCE IF NOT EXISTS ltdegbdm.MyTable
ALTER SEQUENCE ltdegbdm.MyTable_id_seq OWNED BY ltdegbdm.MyTable.id
ALTER TABLE ltdegbdm.MyTable ALTER COLUMN id SET DEFAULT nextval('ltdegbdm.MyTable_id_seq')

I looked up the sequence DDL:
CREATE SEQUENCE ltdegbdm.MyTable_id_seq
	INCREMENT BY 1
	MINVALUE 1
	MAXVALUE<PHONE_NUMBER>854775807
	START 1
	CACHE 1
	NO CYCLE;

which apparently uses default values. So the sequence starts at 1 and not at 100.
Liquibase 3.10.3, PostgreSQL 12

Hi @MahatmaFatalError Thanks for the additional context.  How were you able to get the generationType="ALWAYS" to get past the DTD errors?
This use case is still a problem but I couldn't recreate this with that attribute on the addAutoIncrement changetype
https://github.com/liquibase/liquibase/blob/54e3b592aa13221626d7ee704037345bfa9cabb6/liquibase-core/src/main/resources/www.liquibase.org/xml/ns/dbchangelog/dbchangelog-4.1.xsd#L551

in dbchangelog-3.8.xsd there is no dtd error, although I can't find generationType either 🤔
    <xsd:element name="addAutoIncrement">
        <xsd:complexType>
            <xsd:attributeGroup ref="changeAttributes"/>
            <xsd:attributeGroup ref="tableNameAttribute"/>
            <xsd:attribute name="columnName" type="xsd:string" use="required"/>
            <xsd:attribute name="columnDataType" type="xsd:string"/>
            <xsd:attribute name="startWith" type="xsd:long"/>
            <xsd:attribute name="incrementBy" type="xsd:long"/>
        </xsd:complexType>
    </xsd:element>

I think the original issue was resolved by a different fix at some point. There was a problem where we weren't handling the "sequence data type" setting correctly, and I think that's where the null came from.
BUT, I'll keep this open for testing with #2588 because the generationType/defaultOnNull validation error is fixed there, and the example in the REPO steps use incrementBy/startWith on postgresql which were fixed by that PR also.

@nvoxland moving this to code review since that is where the PR is living now. Thanks!


fix hidden key in info.json

LGTM - thanks for the pr! I actually remove the hidden from the aitools cog since I think it's ready now but thanks!


Option for configuring line length?

For black, we can set the line length in "Python > Formatting: Black Args". Is it possible to have something similar for ufmt?
Thanks!

This is a bit tricky given the way the extension is implemented. Is there a reason that you don't define your preferred line length in a pyproject.toml, where it can be read and obeyed by ufmt/black, regardless of whether it's running from VS Code or CLI?


Issue with X Annotations Disappearing on Zoom in ApexCharts v3 and v4

Hello,
After upgrading ApexCharts from v1 to v3 and later to v4, we’ve encountered an unusual behavior with X annotations when zooming.
Here’s the scenario:
I have a line chart with X-axis values ranging from 00:00 to 23:59 of a single day, divided into 10-minute intervals.
I added an X annotation covering the range from 00:00 to 12:00.
The issue:
When I zoom into a time range that falls entirely within the annotation period (e.g., between 00:00 and 12:00), the annotation’s color disappears.
The only way to retain the color is to include a time range outside the annotation period in the zoom (e.g., from 10:00 to 14:00).
To help reproduce the issue, I’ve created a test repository using version 4.0.1. The same issue was present in version 3.4.0.
Let me know if you need additional details or examples.

chart-test.zip

Hi, I know there has been some fixes in the apexcharts.js code. Could you please check with the latest version blazor-apexcharts 5.0.0


DOC: dialog should be electron.remote.dialog

The code snippet is for the main process, we can probably add a // In the main process. note in it.


[SPARK-48691][BUILD] Upgrade scalatest related dependencies to the 3.2.19 series

What changes were proposed in this pull request?

This PR aims to upgrade:

scalatest to 3.2.19
mockto to 5.12.0
selenium to 4.21
bytebuddy to 1.14.17

Why are the changes needed?

Full release notes:
https://github.com/mockito/mockito/releases/tag/v5.12.0
https://github.com/mockito/mockito/releases/tag/v5.11.0
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Pass GA.
Was this patch authored or co-authored using generative AI tooling?

No.

cc @panbingkun

@wayneguow Could you please manually check the test cases with the @ChromeUITest tag? You can refer to https://github.com/apache/spark/pull/45174

I haven't found the official release notes on https://github.com/scalatest/scalatest/releases yet, let's wait for a while.

I haven't found the official release notes on https://github.com/scalatest/scalatest/releases yet, let's wait for a while.

Yes,  and I will do those test cases mentioned late.

Merged into master. Thanks @wayneguow


implementation of the MAV_CMD_LOGGING_START and STOP messages

I see this mavlink command at pymavlink. Sometimes, these commands are useful. I dont se these command at mavsdk

At some point I started that but didn't finish it, so I removed it again.
Feel free to get inspired and contribute it ;)
https://github.com/mavlink/MAVSDK/pull/1073


auto connect to injected metamask wallet on load

This seems to work as expected on load once the wallet is connected the first time.


Update configure-networking-how-to.md

CORRECTION: the line "When configuring your storage account with private endpoints, public access to your function app will be automatically disabled, and your function app will only be accessible through the virtual network." is not correct.
i have created a whole lab of multiple functions and Storage accounts and added the Private endpoint on the Blob and File but that didn't disable the access to the function from public.
i still have the full lab ready and can discuss it further if needed.

#label:"aq-pr-triaged"

#sign-off

@madsd: I'm sorry - only the author of this article, @ggailey777, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Layered SKGLViews with Xamarin Forms

Description
My application uses 2 different SKGLViews stacked on top of each other to separate the drawing of the background from that of the foreground. This has been working in Android up until version 1.59.3 and stopped in version 1.60. It still works in UWP or if I use SKCanvasView.
Code
Xaml
<RelativeLayout HorizontalOptions="FillAndExpand" VerticalOptions="FillAndExpand">
    <skia:SKGLView x:Name="canvasView"
        PaintSurface="OnPaintSurface"
        RelativeLayout.WidthConstraint="{ConstraintExpression Type=RelativeToParent, Property=Width, Factor=1}"
        RelativeLayout.HeightConstraint="{ConstraintExpression Type=RelativeToParent, Property=Height, Factor=1}"
    />

    <skia:SKGLView x:Name="canvasView2"
        PaintSurface="OnPaintSurface2"
        RelativeLayout.WidthConstraint="{ConstraintExpression Type=RelativeToParent, Property=Width, Factor=1}"
        RelativeLayout.HeightConstraint="{ConstraintExpression Type=RelativeToParent, Property=Height, Factor=1}"
    />
</RelativeLayout>

C#
private void OnPaintSurface(object sender, SKPaintGLSurfaceEventArgs e)
{
    e.Surface.Canvas.Clear(SKColors.Blue);
}

private void OnPaintSurface2(object sender, SKPaintGLSurfaceEventArgs e)
{
    e.Surface.Canvas.Clear(SKColors.Transparent);
}

Expected Behavior
The top view should be rendered transparent and the bottom canvas should be blue.
Actual Behavior
The top view is black and the bottom view isn't visible.
Basic Information

Version with issue:  1.60, 1.60.1
Last known good version:  1.59.3
IDE:  Visual Studio 2015, 2017
Platform Target Frameworks:

Android: 6.0

Target Device:

Samsung Galaxy Tab S

Reproduction Link
VS2015 Solution @ https://www.dropbox.com/s/qnazdsna10wfer0/TestSkia.zip?dl=0

This is interesting. In v1.60.x we changed from a GLSurfaceView to TextureView which should have been a better - in fact, the surface view should not have allowed stacking.
But, since you had it working it appears all my beliefs are crashing down. I will investigate and get back to you as soon as possible.

This is a recurrence of #206 but for the Android platform.
The problem disappears if you set the nativeview transparent with Control.SetOpaque(false) in a custom renderer.

@mgiova thanks for looking into this. I have got a fix.


feat: improve reuse of pubsub/content topic configuration

This is a bug report/feature or change request/support request
Problem

The library consumer has to set the content/pubsub topic in multiple places:

when creating the waku node
when creating an encoder/decoder
when creating a subscription

If the three don't match, then it can lead to unwanted behaviour
Proposed Solutions

Notes

issue description is empty. moving this back to triage

@adklempner could you, please, provide description and some examples of what this feature is about. If possible - link it to relevant epic or milestone

Moving to priority as it is better to finalize before the next release.

@weboko are we set on the proposed solution? do we want to triage it before we tackle it?

todo: comment on design approach, reshare to a feature item for after riliabilit


Refactored CognitoIdentity code examples

Issue #, if available:
Cognito Identity Pool has one code example, list-identity-pools.
Description of changes:
Added:

describe-identity-pool, which displays info about a single pool.
list-pool-identities, which lists the identities in a pool.

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

re-added label for John's changes


Create finite element assembly for Springs elements

Complements the work done in #321, i.e. completes the task description provided in springs element implementation. Specifically, finite element infrastructure was expanded to accommodate Springs elements creation and assembly in OpenTurbine.

@ddement I added unit tests for the residual and stiffness matrix assembly. While these look good, there is something wrong with the contribution to sparse matrix logic -- specifically this r-test is failing due to "KLU2 numeric factorization failed". Could you help with debugging this?


dispatch to multiple repos

What will the user learn in your tutorial?
What should the user already know when starting the tutorial?

Can somebody please review this PR after the latest commit? @denise-khuu


OS X Application UX changes.

@cpauya this fixes #276
Here's the sample screenshot of the preferences menu.

@amodia Pls remove the spinner image on the logo.

@amodia Any screenshot of how it looks like now?

@cpauya here's the current screenshot of the preferences menu.

Good one, merging.

Yayyyyyyyy!


HTTP Error 500.0 - ANCM In-Process Handler Load Failure

Still no resolution for this ?
Should we switch back to using core 2.1 for the time being ?
I added the changes to the Movie class (the Display and Column attributes)
after that I get the above error as soon as I try and navigate to:
https://localhost:44301/Movies.
It is only the Movies route that causes the 500 error, I can't find anything useful by placing breakpoints anywhere.
Various mentions online of web.config, but there IS NO  web.config - only MvcMovie.csproj file and that is only available in the folder view, not in the Solution Explorer for some reason - even with the Show All Files button selected.
From my event log (if it helps):
Failed to start  application '/LM/W3SVC/2/ROOT/movies', ErrorCode  '0x80004005'.
Process Id:  15504.
File Version: 12.2.18292.0. Description: IIS ASP.NET Core Module V2. Commit:
b92c82e97f19f77c95752340066a64a337551e6d
There was another error at the same time:
Only one inprocess application is allowed per IIS application pool. Please assign the application '/LM/W3SVC/2/ROOT/movies' to a different IIS application pool.
(for some reason the github text editor truncates that token value above !!)

Update!!
Ok, hopefully this will help others stuck, because once again vital information was missing from a tutorial.
In VS2017, instead of running the app as IIS Express, I selected MvcMovie from the drop down options available near the green 'run' button.
This appeared to have the effect of selecting the "Project" commandName section of the launchSettings.json file (instead of "IISExpress") which indeed ties in with the tutorial which uses port 5001 (not 51687).
And now the app works fine.
Quite why the app should work properly when debugging from MvcMovie rather than IIS Express, I've no idea - perhaps someone could enlighten me.

?

The assignment of a separate app pool is a requirement when using the in-process hosting model.
That's it

So how is it the app only failes when the user navigates to the movies URL ?

But isn’t in-process hosting the default for IIS ?
On Friday, March 8, 2019, Luis Harvey Triana Vega<EMAIL_ADDRESS>wrote:

The assignment of a separate app pool is a requirement when using the
in-process hosting model.
That's it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/AspNetCore/issues/8134#issuecomment-471003495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKvNVRx0FJnjgRl47jZvlkvP5NklAp_Dks5vUpmOgaJpZM4ba-kr
.

I have got the same, just simply delete IIS config in the launchSettings.json file, run again, and it should work

We're getting same error msg. Application runs fine on IIS (2.2.2 inProcess) for about one week, now it suddenly started getting HTTP Error 500.0 - ANCM In-Process Handler Load Failure. A few minutes later the error dissapeared.
Nothing found in the logs.
#6111

I just started getting this issue from Azure Functions as all of my APIs (in consumption model) are hosted there. Pretty heartbreaking.

 HTTP Error 500.0 - ANCM In-Process Handler Load Failure

 Common causes of this issue:

 The specified version of Microsoft.NetCore.App or Microsoft.AspNetCore.App was not found.

 The in process request handler, Microsoft.AspNetCore.Server.IIS, was not referenced in the application.

 ANCM could not find dotnet.

 Troubleshooting steps:

 Check the system event log for error messages

 Enable logging the application process' stdout messages

 Attach a debugger to the application process and inspect

For more information visit:  https://go.microsoft.com/fwlink/?LinkID=2028526

Drive by Googling for this error led me here...posting my solution in case it can help anyone.
Seemingly out of the blue this issue happened to me. I could run my Core 2.2 API site in Debug in IIS Express, but when I tried to get to my swagger UI endpoint (just /swagger) I'd get the error page. My fix was to delete the VS config folder (a hidden folder in the solution folder), thus deleting the applicationhost.config file. Next time VS built the solution this was recreated, and my issue went away. When I diff the applicationhost.config file between a copy of the deleted one and the new one, I can see what could have caused the error. I had tried to set the default url for my site to be root/swagger, but I did this incorrectly in my project settings. I believe this resulted in the creation of a "swagger AppPool", which is present in the old config but not the new one.

@joegithub - I believe I did the exact same thing you're describing.  One minute my swagger is working great, then I make some edits to the default URL in both project settings and launchSettings.json file, and suddenly no workie anymore.
When you say:  "I did this incorrectly in my project settings" - what is the 'correct' way?  Apparently I caused this issue, and I'd like to avoid it in the future.  Thanks! :-)

@GuitarHero1969 Thanks so much - was looking for ages to see what the issue was and choosing the "Project" instead of IIS express is what solved it.

Thanks so much - was looking for ages to see what the issue was and choosing the "Project" instead of IIS express is what solved it.

This means the app is no longer using IIS. Is that a sufficient answer for everyone on this thread?
I'd like to try and get back to the underlying IIS issue:

Only one inprocess application is allowed per IIS application pool. Please assign the application '/LM/W3SVC/2/ROOT/movies' to a different IIS application pool.

It looks like @joegithub has identified that this issue is due to a corrupt applicationHost.config file. We believe this issue has been fixed in Visual Studio 2019 (cc @shirhatti to follow up).
I'm going to close this as we believe the underlying issue is resolved. If you are having a different issue, please feel free to file a new issue! If anyone is still having the above issue let us know.

In my case, I experienced this error on Azure App Service.
My problem was that I was compiling the web app in x64 because I read in one of the Azure App Service that the OS was 64bits, but what I took long to realize (and it was shown in less detail, but shown nonetheless), was that the IIS PROCESS on Azure App Service was being run in 32 bits. Thus, the assembly was not valid and that caused this error to be shown.
All in all, this particular error message can be the consequence of many different things.
It would be great to have more details instead of such a generic message.

All in all, this particular error message can be the consequence of many different things.
It would be great to have more details instead of such a generic message.

That's being fixed.
cc @jkotalik

Yeah, in 3.0 we have a whole bunch of new detailed error messages. And in Development mode you'll get detailed errors. We're looking at how we can backport some of this to 2.1/2.2.

I wanted to debug on a different URL than the index and I made the mistake to change the "App URL" in project settings from http://localhost:23009 to http://localhost:23009/OtherPage instead of changing the launchUrl  (not the applicationUrl !) in launchSettings.json
It created a bunch of stuff, with a new app pool config for IIS Express and I had this error message after that when I tried to go back to "/OtherPage", after reverting to the default App URL.
I deleted all the references (3 blocks to remove) related to the additional app pool in WebApplication1\.vs\WebApplication1\config\applicationhost.config and it worked again.
Hope this helps. 😄

Well, after one day struggling with the "HTTP Error 500.0 - ANCM In-Process Handler Load Failure" my error was in one http call to another api and an extra dose of stupidity (localhost)..
using (var client = new HttpClient())
{
  client.BaseAddress = new Uri("http://localhost:55945");
   ...
  await client.GetAsync("api/pqp-oh-boi");
}

The way i endup discover my error was through console Kudu. You can access it via Azure or directly by _https://YOUR-SITE.scm.azurewebsites.net/_ . Once in the main page, i went to Debug console/CMD then site/wwwroot and edit the web.config  file.
I set the value of stdoutLogEnabled to true, went back to my directory and access "LogFiles" were I can check for my error.
<aspNetCore processPath=".\Uno.ContactManager.RestApi.exe" stdoutLogEnabled="false" stdoutLogFile="\\?\%home%\LogFiles\stdout" hostingModel="InProcess" />
As @CesarD  said

All in all, this particular error message can be the consequence of many different things.

Hope this can help someone

I closed Visual Studio and deleted hidden .vs folder and now swagger is working again on netcore 2.2

Deleted .vs FTW! Thanks for sharing your solution @joegithub !

It looks like @joegithub has identified that this issue is due to a corrupt applicationHost.config file. We believe this issue has been fixed in Visual Studio 2019 (cc @shirhatti to follow up).
I'm going to close this as we believe the underlying issue is resolved. If you are having a

FYI I am still getting this in Visual Studio 2019 and the solution still works.

Just got the exact same error as @joegithub in VS2019 16.2.3 while adding IdentityServer4 to an ASP core webservice and adding /.well-known/openid-configuration to the project path.
Doesn't appear to be fixed in VS2019 yet.


Update to v10?

Hello, there is any chance that you will update the module to Foundry v10?
Thanks in advance :)

Seconded:  something compatible with V10 would be a very intriguing addition to Foundry's modules.  Lots of potential, here, for both regular and solo games!


Add docker-compose example to README

Hi, I'm new to docker-compose and wanted to use this docker container in my project. I'm a little unsure what that would look like. Here's my best guess, can you advise?
php:
  build: ./php/
  links:
    - composer
# skipping lines . . .

composer:
  build: ./composer/
# etc...

And then inside of ./composer/ a Dockerfile like so
FROM composer/composer

# Use local composer.json file
COPY ./composer.json /var/www/html/composer.json
WORKDIR /var/www/html
RUN composer install

Thanks for the great project(s)!

I was struggling with this, here's the comment that solved it for me:
https://github.com/RobLoach/docker-composer/issues/88#issuecomment-229213544


Sean Killeen Resignation from Maintainers Committee

I appreciate the goal and aims of the committee and the work that its members are trying to do. But its current format and impact does not align with the work I want to be doing in this space, and so I fondly and respectfully bid the committee farewell. This absence will be small, because I have been unable to fully participate for some time anyway due to other priorities and aforementioned constraints.
I remain available for any conversation / discussion which you feel could benefit from my voice. Tag me in at any time.
I wish you every success in your mission of making the .NET Foundation and the .NET ecosystem better for maintainers!

CC @rprouse / @devlead -- sent with all affection and respect. 👍 ❤

While I see no formal reason for you to resign, this is a community effort and we are free to come and go as our lives permit.
I totally respect your wishes, and we're super appreciative of all you contributed to the committee and community ❤

@devlead regarding formal reason -- I was happy to stay dormant, but I wanted to remove my name after the latest handling of the ImageSharp license changes. I don't like the way that DNF and MS showed up there and how it played out, and so I'm choosing to remove my name from association and think about other non-DNF avenues to move things forward.
I know your heads and hearts are generally in the right place in the spirit of things and I wish you success. I intend to explore different ways of working toward the same goals. 👍


If a ticket key callback returns 0 in TLSv1.3 don't send a ticket

If we can't construct the ticket don't send one. This requires a change
to the TLS state machine to be able to a handle a construction function
deciding not to send a message after all.
Fixes #18977
This fix is for master only. It's probably too invasive to be suitable for backport to 3.0/1.1.1. I think I can create a version for those branches without the last commit that introduces the new return type - which should hopefully much restrict the amount of code touched.

Feedback above addressed. Please take another look.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

@mattcaswell This needs a rebase.

@mattcaswell This needs a rebase.

Done. This was trivial, so I assume the existing approvals still apply.

Still OK

Still OK

Ditto.

This pull request is ready to merge

Merged to master. Thank you.


Update changelog.md

ms.date must be updated

@vcarazo - you can't set a date in the future.
@jtarquino @maggiesMSFT
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi, @vcarazo and @jtarquino -- the only change I see is to set the date in the future, which doesn't make sense. I'll sign off when there are actual content changes.

#sign-off

This is the update date I see on this page:
Article · 07/02/2024 · 24 contributors
I dont know the format of this date but 7th feb 2024 or 2nd july 2024 are wrong dates because on September 26, 2024 this page was updated anouncing the release of power bi report server september 2024.

... on September 26, 2024 this page was updated anouncing the release of power bi report server september 2024.

Should the date be set to 09/26/2024 then?


pip3 install crypto may not work

I tested the kflash.py on two devices.pip3 install crypto works on the ubuntu18.04 with python2.7.15rc1.And it doesn't work on the ubuntu 16.04 with python 3.5.2.
In the case of ubuntu 16.04 with python 3.5.2,It doesn't matter if I install crypto or not.I shoud install pycrypto to fix it.

@ly0 I'm wondering if we should just include a pure-Python implementation
of AES128-CBC into kentool.py to get rid of the PyCrypto dependency: the
serial download is very slow so I don't think the encryption performance
matters.
Alternatively, we can do the import only when the user requests firmware
encryption.

@minux Yep, I agree with you, I will replace a pure python implement of AES-128-CBC to remove pycryptodome dependency.

@Aostas Latest kflash.py has removed pycryptodome dependency. :-)


freedomsponsors is not a big bounty platform

freedomsponsors dot org is a gambling website

also:
safehats dot com
bugbountyhq dot com
hackerhive dot io
antihack dot me
aren't reachable


Crashplan for Small Business failed to monitor path(s).

Hello,
I get this message on crashplan launch.
"Real-time file watching failed to be setup on one or more path(s) because Crashplan for Small Business doesnt't have proper permissions."
despite i check the box to launch with high rights in docker.
How can i correct this please ?

You can look at log/service.log.0 under the directory you mapped to /config to find which files are causing the issue.

I see that :
[11.07.19 03:13:06.399 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/Modèles 3D, ignoring traversal [11.07.19 03:13:06.400 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@docker, ignoring traversal [11.07.19 03:13:06.526 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/docker/#recycle, ignoring traversal [11.07.19 03:13:07.208 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/homes/<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:07.277 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/homes/zabouchon/.Maildir, ignoring traversal [11.07.19 03:13:07.293 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/homes/admin/.Maildir, ignoring traversal [11.07.19 03:13:12.826 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/video/<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:15.800 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/Scan, ignoring traversal [11.07.19 03:13:15.917 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:15.918 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:15.919 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@database/pgsql, ignoring traversal [11.07.19 03:13:16.213 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/photo/<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:53.204 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@calendar/attach, ignoring traversal [11.07.19 03:13:54.595 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/z, ignoring traversal [11.07.19 03:13:54.595 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/Sauvegardes, ignoring traversal [11.07.19 03:13:54.595 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/Download, ignoring traversal [11.07.19 03:13:56.773 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/surveillance/#recycle, ignoring traversal [11.07.19 03:13:57.219 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/surveillance/Logs/<IP_ADDRESS>, ignoring traversal [11.07.19 03:13:57.219 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/surveillance/Logs/<IP_ADDRESS>, ignoring traversal [11.07.19 03:13:57.243 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/chat, ignoring traversal [11.07.19 03:13:57.245 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@tmp/nginx, ignoring traversal [11.07.19 03:13:57.246 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:57.248 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@download/pyload/tmp, ignoring traversal [11.07.19 03:13:57.248 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@download/pyload/Downloads, ignoring traversal [11.07.19 03:13:57.248 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@download/pyload/Logs, ignoring traversal [11.07.19 03:13:57.249 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@iSCSI, ignoring traversal [11.07.19 03:13:57.249 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/Sources, ignoring traversal [11.07.19 03:13:57.250 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/lost+found, ignoring traversal [11.07.19 03:13:57.250 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@autoupdate, ignoring traversal [11.07.19 03:13:57.283 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/music/<EMAIL_ADDRESS>ignoring traversal [11.07.19 03:13:57.386 WARN  Thread-43    ode42.jna.inotify.InotifyManager] Unable to add watch for path /volume1/music/Best of Emotional Music, errno: , 13 [11.07.19 03:14:01.173 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@appstore/DNSServer/named/etc/key, ignoring traversal [11.07.19 03:14:02.955 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@appstore/AntiVirus/etc/debugdat, ignoring traversal [11.07.19 03:14:04.334 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/@MailScanner/incoming/SpamAssassin-Temp, ignoring traversal [11.07.19 03:14:04.356 INFO  Thread-43    e42.jna.inotify.JNAInotifyWorker] Do not have read permissions for file /volume1/Cloud, ignoring traversal
But i don't know why it happen now and not before, and how i can correct it.
Because docker run with high rights.

Maybe these are new files created with different ownership/permissions?  ls -l /path/to/file will show you the ownership and permission of the file.
Also, running the container with high rights is not required and won't fix the issue.  The user/group   USER_ID and GROUP_ID you configured for the container doesn't seem to have permission to read these files.

I have done a ls -l on every folder and on some folders where everything is ok.
@eadir and #recycle have root/root but everything else was created and owned by bagu/users
USER_ID and GROUP_ID are 1000 and 1000 (don't remember why i've set it to 1000)
User bagu are 1026/100
User root are 0/0
For tests purpose, i set USER_ID to 0 and GROUP_ID to 0 and there is no error until now (will continu to test)

USER_ID and GROUP_ID of 1000 are the default values.

If you manage to find a fix outside of setting it for root, please report back. I have instances where some files and folders created by the admin user on my Synology (not root, and not the default admin) have no problems at all, while other files and folders created by the same user have watch problems. CrashPlan is set for that same user and group, so I'm not sure why some have issues while others do not.

Do not know if this is a same issue u have or different. Had the same message, you just described. Apparently, the docker was set up as the "admin" user, but i did login to the crashplan web interface as a different user, within "Administrators" group.
I did notice while browsing through the folders, to choose what items to back up and what items to skip, that I cannot see some of the folders, but i knew i should have an access via "Administratros" grooup, as soon as I gave explicit access to my user to the folder, then i saw the folder from Crashplan interface instantly and no more messages with the error


Error: Cannot find module 'csjs-inject' from ' ... /remix/src/ui'

I want to run remix on my localhost but I get this error! any suggestion ? My browser show nothing and I see this error in my browser console:
index.html:10 Uncaught TypeError: Cannot read property 'ui' of undefined
    at loadDebugger (index.html:10)
    at onload (index.html:6)

Did you run npm install?

Yep! I did and npm install return the
Error: Cannot find module 'csjs-inject' from ' ... /remix/src/ui'
And did some other stuff.

I think I had the same problem. Csjs is not listen in package.json. Try to install it separately
npm install csjs-inject and I think it should work then.


[JENKINS-26156] Fixing BodyInvoker.withDisplayName Javadoc

See https://github.com/jenkinsci/workflow-cps-plugin/pull/3 for discussion.
@reviewbybees

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

:bee:


FEAT: Add IPFS / Pinata avatar upload

IPFS Upload and resolution via Pinata

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

@zencephalon yeah we got https://cloudflare-ipfs.com


ByteString introduced as AsciiString super class

Motivation:
The usage and code within AsciiString has exceeded the original design scope for this class. Its usage as a binary string is confusing and on the verge of violating interface assumptions in some spots.
Modifications:

ByteString will be created as a base class to AsciiString. All of the generic byte handling processing will live in ByteString and all the special character encoding will live in AsciiString.

Results:
The AsciiString interface will be clarified. Users of AsciiString can now be clear of the limitations the class imposes while users of the ByteString class don't have to live with those limitations.

@trustin @nmittler @buchgr - FYI.

@netkins build

@Scottmitch looks great overall! Just a few questions/comments.

@nmittler - Thanks for review! I think I got all your comments...feel free to take a look.

Thanks @Scottmitch! Will review today.

@trustin - Sounds good.  Ping me when you are finished.

@Scottmitch Review finished. Perhaps we don't need AsciiString anymore and we should move textual operations to a utility method? See here.

Moving ByteString, AsciiString and ByteVisitor to netty-common (since it's not limited to writing a codec anymore)
@trustin - We may be creating a circular dependency here. ByteString exposes methods that take ByteBuf types...and so we would have to add netty-buffer as a dependency of netty-common but the netty-buffer package already depends upon netty-common. We may have to introduce an abstraction layer...or move some other stuff....any suggestions?

Those methods are just shortcut methods. We could move them to ByteBufUtil?

Review done. Please ping me for another round. Thank you for your patience!

Moving ByteString, AsciiString and ByteVisitor to netty-common (since it's not limited to writing a codec anymore)

@trustin - We may be creating a circular dependency here. ByteString exposes methods that take ByteBuf types...and so we would have to add netty-buffer as a dependency of netty-common but the netty-buffer package already depends upon netty-common. We may have to introduce an abstraction layer...or move some other stuff....any suggestions?

Those methods are just shortcut methods. We could move them to ByteBufUtil?

Good idea. I had to manually do some conversions which I was leaning on ByteBuf to do for me.  Please review.  I'm wondering if I should worry about manually releasing the ByteBuffer object I allocate to do the conversion...

@trustin - Ready for another round.

@Scottmitch Looks great! Please cherry-pick once you make the getBytes() methods private. If you are inclined to making them public, we could move it somewhere more appropriate.

@trustin - Methods made private for now.  As you said we can always increase visibility later.  I'll pull this in after https://github.com/netty/netty/pull/3626#discussion_r28297248.

@buchgr - FYI.  I'm going to pull this in after your #3626 PR.  This may cause a bit of heart-ache for your headers optimization work, but hopefully not much if you are focused mostly on DefaultHeaders.

@Scottmitch thanks! Will update once I get to work!

PR https://github.com/netty/netty/pull/3631 will serve to review the forward-port into master.

Cherry-picked into 4.1 (9a7a85d and e36c143)


Verification of flash failed - STM32WB55RG

[X] Programmer/board type:Stlink/v2.1 onboard
[ ] Programmer firmware version: e.g STSW-LINK007 2.27.15
[X] Operating system: Ubuntu 18.04
[X] Stlink tools version and/or git commit hash: v1.5.1-30-g84f63d2
[X] Stlink commandline tool name: st-flash
[X] Target chip (and optional board): STM32WB55RGV6

If I compile a program that exceeds 8192 bytes (2 pages) on my STM32WB55RG board, I receive the error below. If I keep it below 8192 bytes, it flashes just fine. So, apparently, something is happening at the end of page 2 or beginning of page 3.
I've tried running st-flash erase prior to flashing, but it makes no difference. I've also tried multiple resets and power cycles.
The programs (both working and failing) are compiled via a make file that was generated with cubemx. The program that fails to flash is the default setup generated by cubemx, without any additional code.
I have a L476RG board that I've flashed programs in the 500KiB range using this version of st-flash. No errors were produced in that case.
**Edit: I just tried to flash a pre-compiled binary from ST for this board, and I got the same error. It appears to write all the data (per the cli), but errors when verifying at 8192.
Output:
st-flash write WB55_02.bin 0x8000000
st-flash 1.5.1-30-g84f63d2
2019-06-16T13:58:49 INFO common.c: Loading device parameters....
2019-06-16T13:58:49 INFO common.c: Device connected is: WB55 device, id 0x20016495
2019-06-16T13:58:49 INFO common.c: SRAM size: 0x40000 bytes (256 KiB), Flash: 0x100000 bytes (1024 KiB) in pages of 4096 bytes
2019-06-16T13:58:49 INFO common.c: Attempting to write 11612 (0x2d5c) bytes to stm32 address: 134217728 (0x8000000)
Flash page at addr: 0x08002000 erased
2019-06-16T13:58:49 INFO common.c: Finished erasing 3 pages of 4096 (0x1000) bytes
Writing
Starting   2 page write
2019-06-16T13:58:50 INFO common.c: Starting verification of write complete
2019-06-16T13:58:50 ERROR common.c: Verification of flash failed at offset: 8192
stlink_fwrite_flash() == -1

Expected/description:
successful flash

@section14 Hey, any luck with that? I also summon the @WRansohoff since they are the author of STM32WB integration.

I think this issue can now be closed @section14 as I have merged the fix from @eupn. Could you verify?

Oh, thank you for fixing this - it looks like it might have been a similar issue as #797? Sorry about that, and sorry that I missed the ping.

Unfortunately this fix does not work for me, I keep getting :(
ERROR common.c: Verification of flash failed at offset: 0
stlink_fwrite_flash() == -1

Obviously fixed by #816.