Losing exceptions generated within Stored Procedures in MS SQL Server 2012

Hi, I have a strange one, whereby I seem to be losing exceptions from MS SQL Server.
When I invoke a stored procedure, using the 'ProofOfPyodbcError.py' (attached), I receive exceptions up until I do some sort of Update DDL in the Stored Procedure.
Prior to the Insert/Update/Delete, I can receive exceptions.
The ProofOfPyodbcError.py script has hard coded Database connection string, so it will need to be tweaked if you wish to run it.
After the Insert/Update/Delete, it doesn't matter what I do, no exceptions are received by the python code.
The attached Proof_Of_Error_DDL.sql will establish a test table and stored procedure, although, the script does have a hard coded Database name in it.
The ODBC settings are done using C:\Windows\SysWOW64\odbcad32.exe to create a "System DSN", which is used in the Database connection string.
Am I simply doing something wrong with my Python code or is there something deeper?
I originally hit this whilst using SQLAlchemy but have narrowed it down to pyodbc.
Environment:
Windows 7 Microsoft Windows NT 6.1 (7601) 64 bit
Python 2.7.9 (default, Dec 10 2014, 12:24:55) [MSC v.1500 32 bit (Intel)] on win32
pyodbc-3.0.10.-cp27-none-win32.whl
Microsoft SQL Server Enterprise (64-bit) 2012 (11.0.3128.0) Unclustered, stand alone on a development workstation

Suggestions more than welcomed!

        END TRY
        BEGIN CATCH
            --print 'Exception whilst executing delete'
            set @anInteger = 'UNEXPECTED DELETE ERROR';
        END CATCH;
    END
    ELSE
    BEGIN
        --print 'Did not Execute Delete'
        set @anInteger = 'DID NOT EXECUTE DELETE';
    END;
    -- print 'exception regardless of which path taken'
    set @anInteger = 'ERROR REGARDLESS OF PATH';

    --print 'Returning @anInteger = '+cast(@anInteger as nvarchar)
    return @anInteger
END
GO

Proof to show that SQL Server does raise exceptions when expected, using the execution mechanism as captured by SQL Trace.
use mssql_db_dev
declare @p1 int

BEGIN TRY
print 'Running Delete'
set @p1=NULL
exec sp_prepexec @p1 output,
N'@P1 int',
N'
declare @result int;
EXEC @result = spProofOfError @P1;
select @result as result;
',1;
END TRY
BEGIN CATCH
    SELECT 'With Delete'
        ,ERROR_NUMBER() AS ErrorNumber
        ,ERROR_SEVERITY() AS ErrorSeverity
        ,ERROR_STATE() AS ErrorState
        ,ERROR_PROCEDURE() AS ErrorProcedure
        ,ERROR_LINE() AS ErrorLine
        ,ERROR_MESSAGE() AS ErrorMessage;
END CATCH;

BEGIN TRY
print 'Without running Delete'
 set @p1=NULL
 exec sp_prepexec @p1 output,
 N'@P1 int',
 N'
 declare @result int;
 EXEC @result = spProofOfError @P1;
 select @result as result;
 ',
 0;
END TRY
BEGIN CATCH
    SELECT 'Without Delete'
        ,ERROR_NUMBER() AS ErrorNumber
        ,ERROR_SEVERITY() AS ErrorSeverity
        ,ERROR_STATE() AS ErrorState
        ,ERROR_PROCEDURE() AS ErrorProcedure
        ,ERROR_LINE() AS ErrorLine
        ,ERROR_MESSAGE() AS ErrorMessage;
END CATCH;

Python code using pyodbc to invoke the above stored procedure, expecting the same results as manually executing the stored proc in SQL Server Management Studio
from __future__ import print_function
import logging
logging.basicConfig(level=logging.DEBUG)

#Default logging, overridden by command line option right down the bottom of this file
logger = logging.getLogger("ProofOfError")
logger.setLevel(logging.DEBUG)

import pyodbc
print("pyodbc version = ", pyodbc.version)
logger.debug("Attempting connection to ***DEV WORKSTATION*** SQL Server for info (hard coded credentials for ODBC PFdb)")

db = pyodbc.connect('DRIVER={SQL Server};SERVER=localhost;DATABASE=mssql_db_dev;UID=db_user_dev;PWD=dev')
logger.debug("Connection succeeded")

executeStoredProcedureSQL = """
declare @result int;
EXEC @result = spProofOfError ?;
select @result as result;
"""
#executeStoredProcedureSQL = """select count(1)+1 from dbo.proof_of_error;"""
logger.debug("SQL = '%s'", executeStoredProcedureSQL)

try:
    logger.info('\n\nRunning WITHOUT DELETE')
    cursor = db.cursor()
    cursor.execute(executeStoredProcedureSQL, (0,))  # zero does not run the DELETE
    logger.debug("cursor.description = ", cursor.description)
    if cursor.description:
        row = cursor.fetchone()
        if row:
            logger.debug("column[0] = %s", repr(row[0]))
        else:
            logger.debug("no rows returned")

except Exception as ex:
    logger.error("exception ignored - "+repr(ex))
finally:
    if cursor:
        cursor.close
    else:
        logger.debug("Cursor None")

try:
    logger.info('\n\nRunning WITH DELETE')
    cursor = db.cursor()
    cursor.execute(executeStoredProcedureSQL, (1,))   # 1 runs the DELETE
    logger.debug("cursor.description = %s", cursor.description)
    if cursor.description:
        row = cursor.fetchone()
        if row:
            logger.debug("column[0] = %s", repr(row[0]))
        else:
            logger.debug("no rows returned")

except Exception as ex:
    logger.error("exception ignored - "+repr(ex))
finally:
    if cursor:
        cursor.close
    else:
        logger.debug("Cursor None")

I notice that your stored procedure does not include
SET NOCOUNT ON;

as the first executable statement. Doing so could very likely avoid the issue.

I'm experiencing a similar issue. When pyodbc executes the following stored procedure, the error (divide by 0) is not caught
Here is the stored proc
IF OBJECT_ID('dbo.spSwallowError') IS NULL
EXEC ('CREATE PROCEDURE dbo.spSwallowError AS SET NOCOUNT ON;');
GO
ALTER PROCEDURE dbo.spSwallowError
AS
BEGIN
SELECT 1;
SELECT 1 / 0;
 END;

@githubmlai - It looks like your ALTER PROCEDURE is clobbering the SET NOCOUNT ON; from the CREATE PROCEDURE. Try this instead:
ALTER PROCEDURE dbo.spSwallowError
AS
BEGIN
    SET NOCOUNT ON;
    SELECT 1;
    SELECT 1 / 0;
END;

@githubmlai - The SELECT 1; statement has already put a result set into the "pipeline" back to the client, so you won't see the exception until you call nextset() to try and retrieve the second result set:
>>> import pyodbc
>>> conn = pyodbc.connect("DSN=myDb;")
>>> crsr = conn.cursor()
>>> crsr.execute("{CALL dbo.spSwallowError}")
<pyodbc.Cursor object at 0x03041218>
>>> rows = crsr.fetchall()
>>> rows
[(1, )]
>>> crsr.nextset()
Traceback (most recent call last):
  File "<pyshell#6>", line 1, in <module>
    crsr.nextset()
pyodbc.DataError: ('22012', '[22012] [Microsoft][SQL Server Native Client 10.0][SQL Server]Divide by zero error encountered. (8134) (SQLMoreResults)')

Thanks!  Any suggestions on how to guarantee pyodbc rethrows the error? Should I always fetch all the rows and then call next set repeatedly?

@githubmlai - Essentially, yes. You'd just keep calling nextset() until it returned False. The exception, if any, would either get thrown immediately or on one of the nextset() calls.

ERROR_NUMBER() AS ErrorNumber
ERROR_SEVERITY() AS ErrorSeverity
ERROR_STATE() AS ErrorState
ERROR_PROCEDURE() AS ErrorProcedure
ERROR_LINE() AS ErrorLine
ERROR_MESSAGE() AS ErrorMessage;
how to get these parameters in pyodbc using python? can anyone help me out please!!!!


Compile only certain contracts

Use case:
I have several contracts in my project. Some of them are already deployed and used in production.
Project
└───contracts
│   └───old-contracts
│   |    contract-1-v-0.2.4.sol
│   |    contract-2-v-0.2.4.sol
│   └───new-contracts
│        contract-1-v-0.5.1.sol
│        contract-2-v-0.5.1.sol
│
└───deployment
    │   deploy.js
|
└───test

Currently there is no way I can specify only the old-contracts to be compiled.
It would be nice to have a feature so that I can specify a certain contract that I want to compile. Compiling whole folder folder would be nice to.
for e.g:
etherlime compile old-contracts --solcVersion=0.2.4

etherlime compile old-contracts/contract-1-v-0.2.4.sol --solcVersion=0.2.4

etherlime compile new-contracts --solcVersion=0.5.1

This issue is already fixed. You can use these both optional params:
buildDirectory - [Optional] Defines the directory for placing builded contracts.
workingDirectory - [Optional] Defines the folder to use for reading contracts from, instead of the default one: ./contracts. Here can be specified also a single solidity file for compiling e.g: /contracts/LimeFactory.sol.


Improve bioschema in nmrXiv- add more properties

Project:

[ ] Citation
[ ] projectDomain -> variable
[ ] projectSubject -> variable
[ ] Creator
[ ] about
[ ] studyProcess?? (protocols)

Study

[ ] Citation
[ ] studyDomain -> variable
[ ] studySubject -> variable
[ ] Creator
[ ] about
[ ] studyProcess?? (protocols)

Dataset

[ ] Citation
[ ] Keywords
[ ] Creator
[ ] Add molecules
[ ] Add nucleus

Sample

[ ] rdf:type
[ ] url
[ ] Add solvent

Molecule

[ ]  url to pubchem/chebi
[ ] iupacName
[ ] Delete DOI

Discuss:

[ ] LabProtocol
[ ] ChemicalSubstance
[ ] Journal


Stuck on version 0.7

Description
Hi, I can't update the sam version to v 0.10
Steps to reproduce
sudo pip install aws-sam-cli
sam --version
SAM CLI, version 0.7.0

OS: Ubuntu
sam --version: 0.7

You might have installed it initially with the ‘—user’ flag (our docs state to use that flag so it’s likely what you did).

Don't remember.. so what will be the solution?

It would be to add the ‘—user’ flag.

With this was the solution:
pip install --user --upgrade aws-sam-cli

there are easier installation methods than pip. Pip is an advanced installation method.
Refer: https://aws.amazon.com/serverless/sam/


Provide configuration via Liferay Control Panel

Actually every change of the plugin configuration including switching the plugin on/off requires re-reading portal-ext.properties, which in turn requires restart of Liferay.
It would be extremely useful to have this plugin configurable via Liferay Control Panel with immediate result.
This would require a Liferay JSP hook for Control Panel and some further management code.

Thanks for this suggestion. Indeed a good idea. Probably not with a JSP hook but a proper Control Panel Portlet, but that's a detail.

About to work on this, together with #12...

... and LR62? (I know that LR62 is quite like a zombie ... but due to certain license politics it is still in use on many sites ...)

Due to the fact that both the configuration backend and the portlet-setup is different between 6.2 and 7.0, it would be a complete duplicate implementation between the two. This holds me from implementing the 6.2 control panel configuration.
Maybe at some point in the future, but for now I accept that 6.2 users have a sub-optimal experience... (but you're free to convince me anyway :-) )

It was just a kind of hope :-/ Actually I do not think that I will find time to do it by myself however.


Comment map key positions are incorrect in some cases in swc_ecmascript 0.29.0

Describe the bug
Had a lot of tests fail. Probably related to this statement: https://github.com/swc-project/swc/pull/1527/files#diff-ff99e374b96f1bc47fba4a477800d8f6e78d43ddb8d4899afb6b7d9073e7139cR281
Given the following code:
while (1) {
/* test */}

swc now has the following leading and trailing comment maps:
Leading: {BytePos(23): [Comment { kind: Block, span: Span { lo: BytePos(12), hi: BytePos(22), ctxt: #0 }, text: " test " }]}
Trailing: {}

Byte position 23 is after the close brace token and not before it.
Expected behavior
Leading: {BytePos(22): [Comment { kind: Block, span: Span { lo: BytePos(12), hi: BytePos(22), ctxt: #0 }, text: " test " }]}
Trailing: {}

Another Scenario
Actually, I think this has the same cause.
switch (1) {
    case 2:
        3;
    // 4
}

Now (incorrect):
Leading: {BytePos(46): [Comment { kind: Line, span: Span { lo: BytePos(40), hi: BytePos(44), ctxt: #0 }, text: " 4" }]}
Trailing: {}

Previously (correct):
Leading: {BytePos(45): [Comment { kind: Line, span: Span { lo: BytePos(40), hi: BytePos(44), ctxt: #0 }, text: " 4" }]}
Trailing: {}

Version
swc_ecmascript 0.29.0

I'll implement a test system for comments.

Fixed and published a new version :)

Thanks @kdy1! It is working perfectly now 😊


made region a required argument, used the appropriate operator on som…

…e checks and removed boto3 as it's a third layer dependency brought in by the library.

Nice!


tandoor-recipes: Test setup of script name setting

Description of changes
Verifies that all href attributes emitted as part of the login page point to the correct path underneath the script name. I'm not sure whether it's broken because of an upstream bug or missing configuration.
Hopefully helps with #262857.
@ofborg test tandoor-recipes-script-name
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg  test tandoor-recipes-script-name

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/how-to-serve-tandoor-recipes-from-a-sub-path/32451/2

@ambroisie Welp, after actually testing what I created this PR for, it looks like the very first URL on the page is broken:
Verify that each <a> element's @href URL can be reached: ['http://localhost:8080', '/any/path/search/', '/any/path/plan/', '/any/path/shopping/', '/any/path/books/', '/any/path/accounts/login/', '/any/path/accounts/password/reset/']
machine: must succeed: curl --cookie /tmp/cookies.txt --cookie-jar /tmp/cookies.txt --fail --header 'Origin: http://localhost:8080' --show-error --silent http://localhost:8080
machine # [  201.564197] gunicorn[1391]: [2024-06-19 10:17:23 +0200] [1391] [ERROR] Error handling request /
machine # curl: (22) The requested URL returned error: 500
machine # [  201.568333] gunicorn[1391]: Traceback (most recent call last):
machine # [  201.568799] gunicorn[1391]:   File "/nix/store/7abksi7h0b8nwza1574fmfg0fmb4jcf0-python3.11-gunicorn-21.2.0/lib/python3.11/site-packages/gunicorn/workers/sync.py", line 135, in handle
machine # [  201.569534] gunicorn[1391]:     self.handle_request(listener, req, client, addr)
machine # [  201.569923] gunicorn[1391]:   File "/nix/store/7abksi7h0b8nwza1574fmfg0fmb4jcf0-python3.11-gunicorn-21.2.0/lib/python3.11/site-packages/gunicorn/workers/sync.py", line 168, in handle_request
machine # [  201.571401] gunicorn[1391]:     resp, environ = wsgi.create(req, client, addr,
machine # [  201.571816] gunicorn[1391]:                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
machine # [  201.573417] gunicorn[1391]:   File "/nix/store/7abksi7h0b8nwza1574fmfg0fmb4jcf0-python3.11-gunicorn-21.2.0/lib/python3.11/site-packages/gunicorn/http/wsgi.py", line 183, in create
machine # [  201.574003] gunicorn[1391]:     path_info = path_info.split(script_name, 1)[1]
machine # [  201.574464] gunicorn[1391]:                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
machine # [  201.574817] gunicorn[1391]: IndexError: list index out of range


[Discussion]: Can we leverage WASM to improve the image tool?

I think we can, but I'm not entirely sure, hoping someone with more experience in WASM than myself can chime in.

You definitely can. There is a WASM port of ImageMagick available, and libsquoosh which powers squoosh.app. Squoosh has good support for AVIF #44 😉

Ouch, about Squoosh, it seems web support is not quite there yet 😢 GoogleChromeLabs/squoosh#1084

@gaston-flores fantastic! No worries about Squoosh, looks like ImageMagick supports avif, so I would presume the WASM port does as well 🤞

Ah, just kidding, no avif - https://cancerberosgx.github.io/demos/WASM-ImageMagick/supported-formats/

Damn 😂 Sharp is also working on WASM support here lovell/sharp#3336, so maybe we just need to wait 🤞

@gaston-flores I think you're right, it's just not out there yet, I'll keep this open in the mean time though. Thanks for the info!


[BEAM-905] Add shading config to examples archetype and enable it for Flink

R: @davorbonaci AND @aljoscha
This makes the Flink quickstart work out of the box.

LGTM!
The quickstart instructions for the Flink runner would not actually be working without this. When I wrote them I must have copied the shading part from the java examples pom to my generated word count example POM.

LGTM. (Please self merge.)

Thanks both!


request to be made a maintainer/owner using Jenkins ID

@rmkeeble -<EMAIL_ADDRESS>is aware of this request

Description

Submitter checklist for changing permissions

Always

[ ] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] All newly added users have logged in to Artifactory at least once

All ok to add Chris

Why are you filing duplicate pull requests?

There were some mistakes.
Should be adding chris_overend to vectorcast-execution and vectorcast-coverage plugins.
Cheers
Richard
From: Daniel Beck<EMAIL_ADDRESS>Sent: Wednesday, September 26, 2018 4:57 PM
To: jenkins-infra/repository-permissions-updater<EMAIL_ADDRESS>Cc: Richard Keeble<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [jenkins-infra/repository-permissions-updater] request to be made a maintainer/owner using Jenkins ID (#870)
Why are you filing duplicate pull requests?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/jenkins-infra/repository-permissions-updater/pull/870#issuecomment-424766584, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVVIOfWSQK5NHTm1MmWXNYoagU4xHG94ks5ue6PjgaJpZM4W6-KB.

I removed incorrect pull requests
Also there are 2 different plugins so I made the changes separately
should these changes be combined?


[RTM] minor bbregister option tweak

Depends on https://github.com/nipy/nipype/pull/1929

I've tested this on a couple of datasets. It seems to help with ds000114 without making any visible negative impact on the beast.

FS6 is already a hard dependency - -hires does not work in 5.3 AFAIK. I would prefer to avoid adding a try catch for partial support of 5.3.

True. Maybe a note in the External Dependencies section of installation.rst, then?

https://github.com/poldracklab/fmriprep/pull/424


Duplicate identifiers

I am not sure whether this is due to gulp-typescript or just typescript, but i cannot get 2 modules compiled like that working together with types whereas preliminary tests with tsc directly work.
Has anybody got a setup working in which 1 module using gulp-typescript is referencing another project 2 also with gulp-typescript and both use tsd types such as node.d.ts ?
I have no problem compiling both and using them, as long as i simply require module 2. But as soon as i use import instead i get into duplicate identifier hell, due to the fact that source files in both modules import e.g. node.d.ts typings from obviously different paths. Classes in both projects use e.g 'streams' or 'lodash' thus both use them in typings and thus use the /// reference syntax.

moved this to SO.. http://stackoverflow.com/questions/36688725/make-typescript-node-modules-work-together


Make _initialize and static _initializers private

We had reports of problems if users added their own _initialize() method.
My initial change was to make the name __initialize and make the method private so at TypeScript would error. But I couldn't find any place that actually overrode _initialize(), so I inlined it into the constructor. We should run this through google3 as a smoke test.
I also made static _initializers private. It was already being renamed in prod.

This looks good, merging!


replace instanceof Array with Array.isArray, fix #255

Thanks for this PR. Would you mind to add a feature detection check? This library is compatible back to IE8. I guess something simple like
return Array.isArray ? Array.isArray(x) : x instanceof Array;

should work

Done

Thanks

Could you publish a new version on npm, please?

Working on it as we speak

Released

:tada: Thanks


Allowing access to the list of claims from the AuthorizedModel

Sometimes there are additional claims that are useful in the context of the application.
Would you consider adding the full list of claims in the AuthorizedModel?
Could be done in multiple ways:
    public class AuthorizedModel
    {
        public string Subject { get; set; }
        public string Name { get; set; }

        // Option 1
        public IEnumerable<Claim> JwtClaims { get; set; }

        // Option 2
        public JwtSecurityToken JwtSecurityToken { get; set; }

        // Option 3 - Or ClaimsIdentity
        public ClaimsPrincipal User { get; set; }

    }

I would be glad to provider a pull request for this if you would consider accespting it.

Hi Sander,
I think this would be a nice feature to add. Thinking it through, it may be a good idea to add a new field in configuration allowing to define which claims must be added to the AuthorizedModel. A star would be all, else a list of comma-separated claim names can be configured. If the configuration field is missing or empty, no claims are added. What do you think?

Hi
Given the following example:
[FunctionName(nameof(HttpTriggerFunction))]
public async Task<IActionResult> HttpTriggerFunction(
    [HttpTrigger(AuthorizationLevel.Function, "get")] HttpRequestMessage req,
    [JwtBinding] AuthorizedModel auth)
{
    if (auth.User.FindFirst(type: "role") != "Admin")
    {
        return new StatusCodeResult(StatusCodes.Status403Forbidden)
    }

    return new OkResult();
}

What would you expect auth.User.FindFirst(type: "role") to do if the user has that role claim, but the developer did not opt-in into the new setting? Is it ok to have the result be false or to throw a NullReferenceException?
Personally, I would not hide this behind a setting. I don't think such a setting would really add something meaningful. The "User" property is only about exposing the data already present. The list of claims is already available in the TokenValidatorService and is already parsed as a ClaimsPrincipal:
https://github.com/nikneem/azure-functions-jwt-binding/blob/8a8eed2796704dff2995aabf2b57b65f7b650a68/src/HexMaster.Functions.JwtBinding/TokenValidator/TokenValidatorService.cs#L50-L57

That is actually a good point you have there, I think adding the claims principal may be a good solution. With the solution to make it configurable, I aimed it to be as lightweight as possible. Now I realize there's no need for iterations through the claims collection, just add the claims principal as a property on the returned model. Love it!

Just added a PR for this one.


It seems that Larasearch do not work with Laravel 5.2?

I am using Laravel 5.2. Today, I need to use ElasticSearch to do full text search. After I required the larasearch, the artisan don't work because in LarasearchServiceProvider::boot, $this->package method is not found. I checked the develop branch and see that it is changed. I update the file manually. After that I tried to generate paths for my models. The config is print in the console. If I specified the --write-config, artisan complainted that config:publish command is not there.
So I think that current released version '^0.8.0' is not compatiable with Laravel 5.2. How can I solve it? I have fork the repository, and if something I can do, I will make a pull requests.

+1


Is it possible to set hosts in an array?

Q
A

Issue Type
Question

Deployer Version
6.2.0

Description
Hi!
Is it possible to set hosts in an array?
$hosts = [
'example.com',
'example2.com',
'example3.com'
];
host($hosts)
->user('ankas')
->port(1322)
...

Yes: host(...$hosts).

O!  thanks


Ignoring sequence numbers (1-6) and id area (73+) columns

Is it possible to add options to ignore these areas, and/or (at the least) an option clear those areas?

We can investigate if there are things we can do (such as clearing "macros"). Generally, the VS Code APIs are limiting the access we have to the underlying Monaco editor to enforce things such as events in specific columns.

Thanks.  I have a python script that does it for me but would prefer to just be able to ignore.  I could see it being painful to code listening for events and handling.

So, if we have a menu option to remove sequence numbers completely from the code, would that be valuable to you?

Yes please, that would be good.  Options to remove sequence numbers (1-6), id (73+), or both.

In the interim, you can hold down ALT+SHIFT and drag the cursor to do a text block selection of the entire sequence number column. Then press DEL and it's gone!

As we are getting closer to launching VS Code as COBOL editor option; we would like to vote for this Enhancement too.

+1

We added the ability to strip and renumber sequence numbers in version 1.2.
https://ibm.github.io/zopeneditor-about/Docs/editor_codechanges.html#renumbering-and-unnumbering-sequence-numbers
@Morridt @cds-ari @bearou @AlexKotsc @FALLAI-Denis let us know what you think.

@phaumer
Hi,
For COBOL, line / sequence numbers are columns 1 to 6... columns 73 to 80 are "ident" (kind of comment on the line).
Columns 1 to 6 can be "numbered" or "unnumbered", but not columns 73 to 80.

Agreed..... "Unnumber Sequence Number" should be 1 to 6.
I see the id area 73 to 80 being cleared instead.
Though the option to clear both the id area and sequence number is still required.
To pinch the "Denumber" option in Topaz Wotrkbench:
O Cols-1-6
o Clear numeric fields only
o Clear all fields
O Cols 73-80
o Clear numeric fields only
o Clear all fields

@FALLAI-Denis @Morridt thanks for the feedback. We focused on customer cases with sequence numbers in 73 to 80, but we should take those other cases under consideration as well. I am reopening this item, but slightly rename it.

@phaumer
hi,
If possible to do it, we would like that sequence number / id area renumber / unnumber menu action should controlled by a settings option.
We use sequence number / id area to « tag » lines for some assistant coding tools (Code generator). Developers must not erase these tags.
Thanks

We updated this capability in v1.2.5 as discussed here. Let us know what you think.


Empty statistics for Document Type "RFCs"

Describe the issue
If you open Statistics and select "RFCs" as Document type, then the result for any option in "Authors" row is empty. For "All" or "Internet-drafts" Document types the statistics is (presumably correctly) displayed.
Code of Conduct

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

I can reproduce only if I also select "past 5 years". Do you see differently? Can you post the URL that you're looking at?
Do you see the histogram or is that empty?

Also remember that /stats is generally broken and needs to be removed/re-implemented (#3712)

For me the behavior is the same for both "All time" and"Past 5 years": the histogram is present, but it is empty. For example, for the following URL
https://datatracker.ietf.org/stats/document/author/documents/?type=rfc
I see the following picture (zoomed out to print full screen):

Marking as under review due to Robert's comment


Auto deleting outdated tmp files

Primitive approach implemented here is to fix outstanding temporary files on Windows - it is especially noticeable for long-running processes temporarily saving some state every couple minutes.

Why are those tmpfiles not already deleted through the regular teardown code?

If I knew why tempfiles remain I would attempt to fix the cause not the unwanted effect.
My use case is very simple - it is the following call utilizing a context manager executed periodically (every couple minutes):

from atomicwrites import atomic_write
..
def save_dump_to_file(self, redis_dump):
     with atomic_write(self._persistence_file, overwrite=True) as f:
          f.write(redis_dump)`

After a day or so there are tens (or maybe hundreds) of tempfiles in the directory of the target file path. Method is provided by a single object and called only by the main thread.
It looks like os.rename (from init.py._move_atomic) sometimes fail and rollback (from init.py._open) may be also failing but any exception raised during the second failure is explicitly masked.
I thought the rollback could be failing because the tempfile is never closed after sync (hence the unnecessary close call added by me)  - but it should be closed as io,open is handled by a context manager.

I understand this is not an elegant way of dealing with not fully diagnosed phenomena. Given the number of project depending on this module and possibly time-consuming cleaning this change adds the behavior should be at least configurable with a kw option in constructor or option configured with a setter to avoid changing the default behavior for anyone not experiencing such issue.

I'm very scared of a program deleting files with a specific suffix. I believe even as a workaround this is undesirable. There's also no point in making this opt-in if there's a bug in atomicwrites that requires you to opt-in.

I gave this a little more thought and, of course, it can absolutely happen that during SIGSEGV/SIGKILL the process terminates while leaking tempfiles. I'd be curious if that's actually the case in your setup. @Girgitt are you running atomicwrites from e.g. celery or something else that has good process isolation where you would not notice segfaults without explicit monitoring?

@untitaker actually I am using atomicwrites within a main thread of a program maintained by a supervisor - nssm on Windows. Each startup of the main thread is logged so segfaults would leave traces.
I solved the problem (at least for my use case) in a (kind of) more elegant way - by trying different approaches to rename tmp file on win32 by trying to use ntfs transaction and then two less-atomic renames if "more atomic" attempts fail. I no longer experience issues with orphaned tmp files however one of my "fallback" strategies can take up to 100ms to complete (https://github.com/Girgitt/python-atomicwrites/blob/more-robust-tmp-file-rename-on-win32/atomicwrites/init.py#L102) which is a definition of bad coding.
The result is probably too complex code for a general-purpose library so I have not issued a pull request. But anyway - if you have 5 min to waste here are changes I made:
https://github.com/untitaker/python-atomicwrites/compare/master...Girgitt:more-robust-tmp-file-rename-on-win32?expand=1
These are not my ideas - rather set of hacks to address limitations of ntfs filesystem I came across when solving similar problems in other projects.

What are those limitations? I feel that even after looking through your changes I still don't understand how you can have dangling tempfiles. Even if the rename failed more often with MoveFileEx than the transactional FS stuff, we should get a reasonable error message that causes us to rollback and delete the file?

Closing due to inactivity, however I am definetly open to using ntfs transactions (even if deprecated) if there are actual problems with MoveFileEx for somebody. I just have to be able to reproduce somehow.


[stable/chart] Add deploymentStrategy on HackMD chart

What this PR does / why we need it:
Add deploymentStrategy on HackMD chart
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] title of the PR contains starts with chart name e.g. [stable/chart]

/assign
/ok-to-test

/lgtm
Thanks for contributing!


Illogical Problem HTML Hint

I use your extension and it's pretty much amazing to have an extension like HTMLHint. However, I always get a strange and illogical problem hint. The problem is shown in the image below. I don't know how come the "$" be not defined!.

I'm guessing HtmlHint won't check that you have jQuery installed (that would be amazing, but I don't know any linters that do that).
You might be able to git rid of this error by adding /* global $ */ as the first line of that script tag
That's a special comment that you can use to tell linters about global variables.


Missing antlr4, Error with az vm image list on ubuntu 20.04 (WSL)

I try to get image list for location francecentral but I received an error.
Command Name
az vm image list
Errors:
No module named 'antlr4'
Traceback (most recent call last):
lib/python3/dist-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
azure/cli/core/commands/__init__.py, ln 528, in execute
    self.commands_loader.load_arguments(command)
dist-packages/azure/cli/core/__init__.py, ln 300, in load_arguments
    loader.load_arguments(command)  # this adds entries to the argument registries
azure/cli/command_modules/vm/__init__.py, ln 31, in load_arguments
    from azure.cli.command_modules.vm._params import load_arguments
azure/cli/command_modules/vm/_params.py, ln 31, in <module>
    from azure.cli.command_modules.monitor.actions import get_period_type
azure/cli/command_modules/monitor/actions.py, ln 7, in <module>
    import antlr4
ModuleNotFoundError: No module named 'antlr4'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az vm image list --location francecentral --publisher Canonical --offer UbuntuServer --sku 18.04-LTS --all -o table

Expected Behavior
See the latest ubuntu lts available for this location
Environment Summary
Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.29
Python 3.8.2
Shell: bash

azure-cli 2.0.81

Extensions:
azure-devops 0.17.0

Additional Context
Same error with different region.
I'm using ubuntu 20.04 on windows 10 on the windows terminal.

hi @qwordy could you pls have a look? we received several issue  on this package. Let's figure out the root cause. thanks.

Are you using the bundled Azure CLI of Ubuntu? The package is old and has some problems. Could you install the latest version and retry?
https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest

@qwordy you're right, the install with one command of the documentation install an old version (2.0.81).
I try to reinstall azcli (v2.7.0) with the manual instructions and it works.
Thanks for your help!

Do you mean curl -sL https://aka.ms/InstallAzureCLIDeb | sudo bash will not install the latest version but an old version?

Yes, I tried it on ubuntu 20.04 and 18.04 (on windows 10 with WSL) and the version installed is 2.0.81 with this curl.

I tried on Ubuntu 20.04 on Windows 10 with WSL and the latest version is installed. Could share your output of this command?
Also @fengzhou-msft
yfy@DESKTOP-ARGPJS4:~$ curl -sL https://aka.ms/InstallAzureCLIDeb | sudo bash
export DEBIAN_FRONTEND=noninteractive
apt-get update
Hit:1 http://archive.ubuntu.com/ubuntu focal InRelease
Get:2 http://archive.ubuntu.com/ubuntu focal-updates InRelease [107 kB]
Get:3 http://archive.ubuntu.com/ubuntu focal-backports InRelease [98.3 kB]
Get:4 http://security.ubuntu.com/ubuntu focal-security InRelease [107 kB]
Hit:5 https://packages.microsoft.com/repos/azure-cli focal InRelease
Get:6 http://archive.ubuntu.com/ubuntu focal-updates/main amd64 Packages [199 kB]
Get:7 http://archive.ubuntu.com/ubuntu focal-updates/main amd64 c-n-f Metadata [5856 B]
Get:8 http://archive.ubuntu.com/ubuntu focal-updates/universe amd64 Packages [110 kB]
Fetched 627 kB in 10s (62.5 kB/s)
Reading package lists... Done
apt-get install -y apt-transport-https lsb-release gnupg curl
Reading package lists... Done
Building dependency tree
Reading state information... Done
curl is already the newest version (7.68.0-1ubuntu2).
gnupg is already the newest version (2.2.19-3ubuntu2).
lsb-release is already the newest version (11.1.0ubuntu2).
apt-transport-https is already the newest version (2.0.2ubuntu0.1).
0 upgraded, 0 newly installed, 0 to remove and 72 not upgraded.
set +v
curl -sL https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > /etc/apt/trusted.gpg.d/microsoft.asc.gpg
set +v
CLI_REPO=$(lsb_release -cs)
echo "deb [arch=amd64] https://packages.microsoft.com/repos/azure-cli/ ${CLI_REPO} main" \
    > /etc/apt/sources.list.d/azure-cli.list
apt-get update
Hit:1 https://packages.microsoft.com/repos/azure-cli focal InRelease
Hit:2 http://archive.ubuntu.com/ubuntu focal InRelease
Hit:3 http://security.ubuntu.com/ubuntu focal-security InRelease
Hit:4 http://archive.ubuntu.com/ubuntu focal-updates InRelease
Hit:5 http://archive.ubuntu.com/ubuntu focal-backports InRelease
Reading package lists... Done
set +v
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  azure-cli
0 upgraded, 1 newly installed, 0 to remove and 72 not upgraded.
Need to get 0 B/49.7 MB of archives.
After this operation, 560 MB of additional disk space will be used.
Selecting previously unselected package azure-cli.
(Reading database ... 31841 files and directories currently installed.)
Preparing to unpack .../azure-cli_2.7.0-1~focal_all.deb ...
Unpacking azure-cli (2.7.0-1~focal) ...
Setting up azure-cli (2.7.0-1~focal) ...

#14011


Break on build. (VRCFury support?)

My avatar uses VRCFury to add such things as clothes, items and so but it seems to be not building VRCFury and then obfuscate the avatar.

Hello
VRCFury is new for me. If it uses additional animators on GameObjects inside the avatar hierarchy, it will not be supported. But I cannot say much about, whitout deeper analysis, what it exactly does and how. I will consider to take a look at it when I work for the next update. But expect it not during this summer.

Sorry for the long time of waiting. I did some research into VRCFury and came to the conclusion that I can't support it.


Better network handling

Hi,
can you modify the switch in such a way that
process.env.NODE_ENV === "development"
would point to
https://dev-wrapmeister.peercoin.net/
and
process.env.NODE_ENV === "production"
would point to
https://wrapmeister.peercoin.net/
?
That way all the cloudflare preview branches will always point to the dev-backend and prod will be prod.

I can also provide custom env variables for dev/prod on CF pages if that helps

This is the documentation btw: https://developers.cloudflare.com/pages/platform/build-configuration/#environment-variables
(So it doesn't get lost in development-bot chatter)


Common menu construction across the application

Right now, all three grid style view controllers construct their own menus, as does the program view controller. It would be great to share this construction across all the view controllers to ensure the menus are consistent wherever the various applicable sections are used.

I'm pretty sure we're now sharing code where it's required. Closing as complete.


Add DMABUF support

To do:

[x] Add additional buffer type (dmabuf in addition to shm buffer)
[ ] (maybe) always create an EGL context independent of backends

Add additional buffer type
Both shm buffers and dmabufs use wl_buffer ids. So windows should just retain a wl_buffer_id field.
However, the container object will become a union of structs.

Useful information: https://www.khronos.org/registry/vulkan/specs/1.2-extensions/man/html/VK_EXT_image_drm_format_modifier.html

Useful information: https://www.khronos.org/registry/vulkan/specs/1.2-extensions/man/html/VK_EXT_image_drm_format_modifier.html


bug in versioning/removed/mockapi.ts case

Request should be correct to versioning/removed/api-version:v2preview/v3 with enumMemberV2Preview. But rejected by mockapi
[ForkJoinPool-1-worker-1] INFO com.versioning.removed.implementation.RemovedClientImpl$RemovedClientService.modelV3Sync - {"az.sdk.message":"HTTP request","method":"POST","url":"http://localhost:3000/versioning/removed/api-version:v2preview/v3","tryCount":1,"Date":"Mon, 30 Sep 2024 02:10:20 GMT","Content-Type":"application/json","x-ms-client-request-id":"5d87308e-055b-45fa-83a4-e375c4f8d42d","Accept":"application/json","User-Agent":"azsdk-java-${project.artifactId}/${project.version} (21.0.3; Windows 11; 10.0)","content-length":45,"body":"{\"id\":\"123\",\"enumProp\":\"enumMemberV2Preview\"}"}
[ForkJoinPool-1-worker-1] INFO com.versioning.removed.implementation.RemovedClientImpl$RemovedClientService.modelV3Sync - {"az.sdk.message":"HTTP response","statusCode":400,"url":"http://localhost:3000/versioning/removed/api-version:v2preview/v3","durationMs":6,"content-length":160,"Date":"Mon, 30 Sep 2024 02:10:20 GMT","Connection":"keep-alive","ETag":"W/\"a0-7pCSm5UxU05anpeENBbPPWfTNCY\"","Content-Type":"application/json; charset=utf-8","redactedHeaders":"Keep-Alive,X-Powered-By","content-length":160,"body":"{\"message\":\"Body provided doesn't match expected body\",\"expected\":{\"id\":\"123\",\"enumProp\":\"enumMemberV1\"},\"actual\":{\"id\":\"123\",\"enumProp\":\"enumMemberV2Preview\"}}"}

com.azure.core.exception.HttpResponseException: Status code 400, "{"message":"Body provided doesn't match expected body","expected":{"id":"123","enumProp":"enumMemberV1"},"actual":{"id":"123","enumProp":"enumMemberV2Preview"}}"

Commented by Crystal, the expected request body should only include id=123, no enumProp as it is removed in v2preview.


Transfer pool init

closes #1085

Coverage increased (+0.4%) to 71.37% when pulling 88593c060058d56bc2a95624c5c33231a8a53dc1 on na3d:transfer-pool-init into f35ac2035f01a5b028a58ef6989181fcde4faeef on linode:develop.

Coverage decreased (-0.1%) to 71.676% when pulling e937b178314c23dd30880870b19eb5a3522cfcab on na3d:transfer-pool-init into 5d698dcbe4e06c69f304315725a68aeb2c338e01 on linode:develop.

Coverage decreased (-0.1%) to 71.672% when pulling 08304ab2f3da41d1c6d3689b11b4163ca0c361a5 on na3d:transfer-pool-init into 5d698dcbe4e06c69f304315725a68aeb2c338e01 on linode:develop.


Missing files at T2_CH_CERN

Sid, Sebastian, please subscribe and report future cases here.

I was going to send this in the email thread, but I'll put it here instead:
The reason I have a suspicion it has to do with WMAgent is because almost all of the files were produced at T2_CH_CERN (and lost at T2_CH_CERN). If it were an SE problem, I'd expect files produced elsewhere to also be affected. Furthermore, EOS admins seem to indicate that for the files they were able to check, there was no record of the file arriving in the namespace. To me, this seems like that the problem occurs even before the file gets to EOS, which then leads me to WMAgent. If I'm thinking about this incorrectly, please let me know.

I'd say the other way around, since it happens only/mostly at T2 CERN, a site/storage issue would be on the top of my list of suspects.
If EOS admins found no records (and logs?) at all for those files, then it probably means the file was not even attempted to be staged out. And in this case, I'd say the job got killed (or killed itself) before the stage out kicked in. And in such cases, the agent wouldn't retrieve a valid FWJR, making it impossible to add the output files to the local database and a posterior phedex injection.
Please do inform us in case you face other files not know by the site and the site admin.

Since we haven't had any issues during the last 11 months, I'm closing this issue


bungie-api-ts/common is not exposed in the generated package.json's exports field

I’m trying to use BungieMembershipType from bungie-api-ts/common and webpack 5 is yelling at me:

Module not found: Error: Package path ./common is not exported from package ./node_modules/bungie-api-ts (see exports field in ./node_modules/bungie-api-ts/package.json)

At the moment I’m digging around to see if I can override the exports field with a plugin. Worst case scenario: I’ll hard-code the enum value. Not the end of the world, but it’s not Technically Correct™.
(Personal opinion: Given that 1. this module only exports ESM code and 2. the structure of the generated code perfectly matches what the exports field contains, the exports field can [again IMO] probably be omitted. One less thing to maintain!)
Thank you for this fantastic package 🙏

Clean solution: I ended up setting exportsFields to an empty array.

Unfortunately your opinion does not match with NodeJS' implementation!
common is reexported from the service-specific modules, so you can do this:
import { BungieMembershipType } from 'bungie-api-ts/destiny2';

Oh perfect, I forgot /destiny2 was a thing. I’ve been relying on editor autocompletion.

Ha, I was about to close this but you beat me to it. Thanks for the speedy response!


build: set GOARCH appropriately

What type of PR is this?
/kind bug
What this PR does / why we need it:
This is necessary for this image to build properly on other architectures.
Requirements:

[x] uses conventional commit messages
[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

/retest

/retest

/assign @tsmetana

/retest

/assign @dobsonj

/assign @jsafrane

/lgtm
/approve


crawler for internshala

Website:
https://internshala.com/
Input:
city
category or keyword

Hey am Interested to work on this can u assign this to me

@iamlordutkarsh  go for it I have assigned to you

Thanks

Will Do pull request tommorow


Update: Office 365 Yammer Android

Office 365 Yammer Android now supports Conditional Access

@nymerias,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@nymerias : Thanks for your contribution to the Azure documentation! The author, @MarkusVi, has been notified to review your proposed change.

Is this going to be updated?

#sign-off


What about 404 HTTP status?

Why API 'not found' has return http status code 200? Could you make a correct 404 http status?

GET /api/articles/BADID HTTP/1.1
...
< HTTP/1.1 200 OK
< Content-Type: application/json
{"flag":false,"message":"Article not found","code":1111}

P.S.: What does it mean '1111' code? Where I can find full list of this codes?
Thank you for excellent generator.

@KIVagant,
you can find a full code list here.
But I will make it to work like send 404 when record is not found.

This issue depends to dingo/api.
Maybe I will close it tomorrow.

Its possible to fall back and use the Symphony standard HTTP response codes. It comes bundled with laravel.
Symfony\Component\HttpFoundation\Response::$statusTexts

Yes, I will change all templates for using dingo/api. So standart HTTP Responses will be there.


Runtime kernel.json files are being left behind on shutdown

Seems like we should clean up the runtime folder kernel.json files. That's likely a result of just killing the process.
Originally posted by @rchiodo in https://github.com/microsoft/vscode-jupyter/issues/1626#issuecomment-1102851926

Is it harmless to remove all these files by mask on exit?
cd /home/username/.local/share/jupyter/runtime
rm kernel-*.json nbserver-*.html nbserver-*.json

Yes they should be removable as long as you don't have any active kernels you're using.

clean up the runtime folder kernel.json files.

From what I can tell we are cleaning these up. Possible its only getting cleaned up in our latest main branch.
However the nbserver-.html & nbserver-.json don't get cleared up as we don't gracefully shutdown jupyter (in non-raw).

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/8827


Tracking issue for os module

https://docs.python.org/3/library/os.html

What's the story here?  I see source code for the OS module but I can't import it.
===== MODULES =====
Traceback (most recent call last):
  File "tests/snippets/whats_left_modules.py", line 8, in <module>
    import os
ModuleNotFoundError: No module named 'os'

Have you set your RUSTPYTHONPATH environment variable to .../RustPython/Lib? Other than that, os is fairly compatible I don't think an issue to track the whole module is necessary anymore.

I have not, I somehow missed that part of the readme.  Thanks!


fix: refactor db private functions

Results in slightly smaller code, plus I don't like that these functions are exposed since they're supposed to be private

eh, 91 bytes is not really worth it if the code is less clear afterwards


/.well-known/webfinger endpoint

The OpenID Connect discovery specification second paragraph is about issue domain discovery based on the RFC7033 webfinger.
Would it be relevant for authlib to provide helpers to implement this endpoint?
The main issue I can see is that this endpoint must be served at the top domain (i.e. mydomain.tld if the IDP is served at auth.mydomain.tld), but this can easily be solved with some webserver proxying configuration.

No. This spec should not be included in Authlib.


Update authorized_keys

add my host liuran.

commit by liuran from china.

Welcome! You should now be able to connect to chur with ssh -p 2020<EMAIL_ADDRESS>and create a private account for yourself.


#1001: made local reports links more flexible (build-snapshot)

… the CI use-case

SonarQube analysis reported 2 issues

 2 info

Watch the comments in this conversation to review them.


Cannot read property 'marginWidth' of undefined

Hi,
After I import the library and insert the appopriate tag inside  app component html, an error pop up in browser console. The error says "Cannot read property 'marginWidth' of undefined" because the widget's opitons seems to be undefined. How can I instantiate a widget properly?
Thank you for your job.

+1

+1

@lucabenedettini did you solve this issue?

No, sorry. @savad

Same problem here just after the installation

The example refers to widget1 in the option attribute. This is the one undefined. Replace it or create it and it will fix the problem.

In the second example, [option]="widget.Item" is not correct. It should be [option]="widget"


No window border after screen resize

Following a screen resize, some applications such as xfce-terminal open without window decorations.
I've seen the same issue after adding a monitor/projector on my laptop.
Logging out and back in resolves the issue.
Running commit 80cdfa8494fcec4fce3b593777788fc1a0fdcaf6.

That almost sounds like Fluxbox crashed and could not be restarted. If this happens again, try running "top" or "ps -auwwx | grep flux" just to verify if that is indeed the problem.

Fluxbox appears to be running fine.  I was seeing fluxbox core files with previous versions, but not anymore.
The problem occurs very consistently on FreeBSD, but only with certain applications.  So far, I've seen it with xfce-terminal and cool-retro-term.
The screenshot here shows applications that were all started AFTER resizing the screen.  Most of them work fine.
Xfce-terminal always gets located in the upper left of the screen and gets stuck at the top layer above all other windows (nothing can overlay it, so I have to close it to get at another window hidden under it).  The menus respond to clicks, but clicking elsewhere on the window doesn't restore focus.  If I go to another window, I have to click on xfce-terminal in the panel to restore focus.
Cool-retro-term does that same thing, but also malfunctions in other ways regardless of screen resizing.  Right now it's showing up as a pink window with no text.  It was working under Lumina a couple weeks ago.

Strange...
Run "xprop" and click on the window. I wonder if those utilities are setting some special "bypass WM" flag on the window, or if Fluxbox just does not know what to do with them...

Same symptoms on PC-BSD, BTW:

Ok, I just reproduced this here as well, and I don't see any of the special "bypass WM" X11 flags on the window, and Fluxbox is still running and handling other windows fine, so you might need to ask the Fluxbox devs about this (or the xfce-termina/retro-cool-term devs, but I think it is most likely a Fluxbox issue).

On 05/12/16 09:55, Ken Moore wrote:

Ok, I just reproduced this here as well, and I don't see any of the
special "bypass WM" X11 flags on the window, and Fluxbox is still
running and handling other windows fine, so you might need to ask the
Fluxbox devs about this (or the xfce-termina/retro-cool-term devs, but
I think it is most likely a Fluxbox issue).
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/pcbsd/lumina/issues/222#issuecomment-218783059

I was able to reproduce this on a VM running just Fluxbox.  I submitted
a PR on the Fluxbox site.
Regards,
 JB

--
All wars are civil wars, because all men are brothers ... Each one owes
infinitely more to the human race than to the particular country in
which he was born.
-- Francois Fenelon

Just bumped into this... as a workaround for now, I just "pkill" fluxbox and then use fluxbox -rc .lumina/fluxbox-int to have fluxbox start as it would from lumina's start and it brings windows decorations back...
Seems to happen more with terminal emulators btw, as I see this mostly with "Sakura" or "gnome-terminal" (not xterm though)


feat(tmux): added tmux plugin installation and upgrade stepes

This PR is included in version 2.8.0 :tada:


⚠ Samsung has degraded performance

In 09c74ea, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3112 ms

Resolved: Samsung performance has improved in 2d391e9.


No "forgot password?" prompt

Hi,
When users fail a login attempt, there doesn't seem to be a prompt for "have you forgotten your password?". According to https://docs.meteor.com/api/passwords.html#Accounts-forgotPassword and https://guide.meteor.com/accounts.html#accounts-ui it seems this should be automatically handles by Accounts UI, is this not the case?
Thanks

Nevermind, had to set
AccountsTemplates.configure({,
showForgotPasswordLink: true,
});


[OneExplorer] Add ConfigObject unit tests

This commit adds more ConfigObject unit tests
ONE-vscode-DCO-1.0-Signed-off-by: dayo09<EMAIL_ADDRESS>
From #1096
For #1051

Coverage Improvement
| coverage (%)
-- | --
20-07-25, main | 29.58%
20-07-25, main | 41.12%
(+12%)


bidi algorithm (for right-to-left Unicode characters)

Unicode provides a bunch of characters, that are needed when using bidirectional text, that is when mixing right-to-left text with left-to-right text. Almost all languages that right-to-left text will need to mix it with left-to-right text on occasion (for example, when adding URLs or numbers).
W3C have this GitHub website that documents the support for the Unicode bidi algorithm in different browsers: https://w3c.github.io/i18n-tests/results/bidi-algorithm

Related:

#996
Via MDN: https://caniuse.com/?search=unicode-bidi

Thanks @Schweinepriester . The issue #996 is about the CSS properties direction and unicode-bidi. Whereas this issue is about the support for Unicode special characters like U+202B. If you read this document explaining these special characters, it says things like:

at the time of writing browsers still don't support RLI/LRI,

It would be useful to know which browsers do support the Unicode bidi algorithm completely, including these special characters.


[WIP] Update form.rst

no need to add symfony/translation as dep. as the symfony/validator has it as requirement
you can use form with Twig without symfony/config and symfony/translation

hmm lot of unrelated tests are failing, is this known?

I've fixed the deprecation notices, so tests should be green again when you update this PR.

I'll re-open when I did the rewrite, thanks!


Add ButtonV2 features in Shadcn Button Component

Is your feature request related to a problem? Please describe.
migrating from ButtonV2 to shadcn Button requires us to use various things various purposes. For example:

AuthorizeButton for authorization
onClick={()=>navigate(url)} for links
shadcn Tooltip component for toolptip

even AuthorizeButton does not support features like variants, refs, className and others.
Describe the solution you'd like
Modify the shadcn button components to handle all the above functions in single place just like ButtonV2. This would improve the developer experience and cleaner code.
@ohcnetwork/care-fe-code-reviewers

I would like to work on this update.

@ohcnetwork/care-fe-code-reviewers
if this issue is valid, assign me

No, we aren't modifying Shadcn components. You don't need to map exact 1:1 to the existing code - point is to get the same functionality, that could be achieved in other ways.


Rationalise pkgs and add qrexec lib and dom0 tools

This changeset clean up support for pkgs, cleaning up the tag finding script.
Also make it possible to create dependencies between packages, using tags.
Also add packages for building qrexec tools, unused in any image right now but useful for future API testing.

Actually forgot to add something. Re-requesting it.


The CustomResourceDefinition "installations.operator.tigera.io" is invalid: metadata.annotations: Too long: must have at most 262144 bytes

Expected Behavior
tigera-operator can work.

Current Behavior
kubectl apply -f tigera-operator.yaml
namespace/tigera-operator unchanged
customresourcedefinition.apiextensions.k8s.io/bgpconfigurations.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/bgpfilters.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/bgppeers.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/blockaffinities.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/caliconodestatuses.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/clusterinformations.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/felixconfigurations.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/globalnetworkpolicies.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/globalnetworksets.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/hostendpoints.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/ipamblocks.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/ipamconfigs.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/ipamhandles.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/ippools.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/ipreservations.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/kubecontrollersconfigurations.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/networkpolicies.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/networksets.crd.projectcalico.org configured
customresourcedefinition.apiextensions.k8s.io/apiservers.operator.tigera.io configured
customresourcedefinition.apiextensions.k8s.io/imagesets.operator.tigera.io configured
customresourcedefinition.apiextensions.k8s.io/tigerastatuses.operator.tigera.io configured
serviceaccount/tigera-operator configured
clusterrole.rbac.authorization.k8s.io/tigera-operator unchanged
clusterrolebinding.rbac.authorization.k8s.io/tigera-operator unchanged
deployment.apps/tigera-operator configured
The CustomResourceDefinition "installations.operator.tigera.io" is invalid: metadata.annotations: Too long: must have at most 262144 bytes

Possible Solution

Steps to Reproduce (for bugs)

Install kubernetes v1.22.5
kubectl create -f https://raw.githubusercontent.com/projectcalico/calico/v3.26.1/manifests/tigera-operator.yaml

That's all.
 What's the intent?
What I mean is, I know where the confusion is coming from. After Kubernetes is installed manually with kubeadm init the success message pops up and gives two steps, the second of which is:
You should now deploy a pod network to the cluster.
Run "kubectl apply -f [podnetwork].yaml" with one of the options listed at:
  https://kubernetes.io/docs/concepts/cluster-administration/addons/

There are suble difference between kubectl create and apply, one of which is that apply allows for easily updating through another 'kubectl apply -f' iteretion. 'kubectl create -f' does not. Doing so throws an error, and the resources have to be destroyed before they can be recreated. This can be disruptive in a production environment, and I don't think that is the intent? Shouldn't the Calico operators be created with apply? What do we do in a production setting when the operators have to be updated?
The fact that create vs apply accepts different metadata annotation lengths is an interesting issue itself. I didn't know that. I'll run that by the k8s project, I'm curious to know more.

Agree with @bluepresleycom regarding production deployment. This (k delete/create) is not something we would prefer in production. Should we expect 3.26.4 with the fix, please.

There are suble difference between kubectl create and apply, one of which is that apply allows for easily updating through another 'kubectl apply -f' iteretion. 'kubectl create -f' does not

If you use kubectl create, you want to use kubectl replace on upgrade (not delete / create).
Apply is the wrong verb for our CRDs. apply is not the same thing as update, and as far as I'm aware the nuanced merging behavior that it carries with it isn't appropriate for our CRDs which should be updated wholesale to exactly the new version.
Happy to be convinced otherwise though, just haven't heard of a use case yet.

This is at its core just a bug in how kubectl apply is implemented. It wasn't design to handle large resources.


Linking to bounty pages via NFT cards

Collected NFT cards do not currently link to their parent bounty page, these should link to the bounty page they are associated with in the same way NFT cards under the “claims” tab do:

Almost perfect but there is a small issue with the text alignment of the "claim id" section, it hugs the left side of the card:


Pretrained Model + Demo

Thank you for a great repo. Will the pretrained model be released somewhere? Also, how can we recreate the video demos shown on the website (https://tsinghua-mars-lab.github.io/DenseTNT/)

We did not release pretrained model since the best performance can be easily reproduced by running our training command. The video demos are created by Bokeh.

Thanks. Given hardware limitations on my end to retrain the entire model, would it be possible for you to release the pretrained weights? It could possibly help others who do not have powerful enough hardware to train the model

We did not release pretrained model since the best performance can be easily reproduced by running our training command. The video demos are created by Bokeh.

Can you provide the code for the demo?

We did not release pretrained model since the best performance can be easily reproduced by running our training command. The video demos are created by Bokeh.

你好，可以提供DenseTNT展览页中演示demo的代码吗？非常感谢


Missing ores in the orbital miner after a few updates ago

We are currently on
AdvancedRocketry-1.10.2-1.1.7
LibVulpes-1.10.2-0.2.2-universal
Currently we are using the ATM fork of the rockhounding mod which has an ore call uninspected mineral ore. It'd doesn't have an oredict by default but I created one with minetweaker ore:oreUninspected. We have this added to the list of ores the orbital miner can use and it WAS working fine about 4-5 updates ago. Players started gradually reporting they werent getting any uninspected ores anymore so after looking into it we can confirm after a few hundred thousand ores mined it hasnt produced a single uninspected now..
About the same time the ore stopped showing up we started getting libvulpes aluminum ore from the miner where we were previously getting thermal expansion aluminum ore so seems like something definitely changed in that regard. Any further insight you may have into this would be helpful. Let me know if you require any additional testing/info on our end.

Couple questions.  Can you verify that the uninspected ore is indeed being
added to the oredictionary in game.  Second, is the oredictionary string
for the ore in the laserdrill ores config option in AR?

yes to both, /mt hand shows it is added to two oredicts oreUninspected and mineralOres
I have both added to the advanced rocketry config and I havent changed anything with the oredicts or advanced rocketry ore config since the pack went public (Nuts and Bolts: Torqued) since the uninspected ore+orbital miner are key points of progression (the ore is unobtainable any other way).
here is the config directly:
https://hastebin.com/akamemucor.coffeescript
I've got alot of stuff added to that list but everything else on the list seems to be working except the uninspected ore. Also the strangeness where we started getting libvulpes aluminum instead of TE a few updates again when this started.

The business with the change in aluminum just sounds like a shift in
ordering of the ores being registered.  Also the config was not designed
with having duplicate entries in mind.  So I'm not even 100% sure what
that'll do.
Also can you post a start up log of the server so I can see if I can spot
anything funky?

What was the last known version that worked?

So there didnt seem to be any way to "weight" the ores, but I found by adding the oredicts multiple times it caused them to appear more often. So the multiple entries exist there to try and create a more weighted mining result.
The last known version.. I'm unfortunately not 100% on which version it stopped working but I do know for sure that AdvancedRocketry-1.10.2-1.1.1 and LibVulpes-1.10.2-0.2.0-universal had everthing generating fine. I'm not sure which update beyond that caused it to stop generating.
Here is a fresh set of all startup logs:
https://drive.google.com/drive/folders/0BwmxXoRAG3ddQ3RaOUhtXzFhU0U?usp=sharing

I don't see anything unusual in the log, I'll grab the pack off of curse and test it today sometime

I downloaded the pack from curse (nuts and bolts: torqued version <IP_ADDRESS>) and loaded it up and built a mining laser on a space station.  I fired it up and got cobblestone, uninspected mineral, resonating ore, cobble, and stone in the first run.

Server/pack is running on forge 2254, I can update to see if that makes a difference later today.
So, things get weirder. Upon seeing you had it working I went into singleplayer and built one and also was able to get uninspected ores. So it's working fine in singleplayer. The ores just arent showing up on the server/in multiplayer. The configs are identical, in fact the ones I linked here earlier were from the server directly.

Later ima see about running a diff on the log output of the server and
single player.  I wonder of its a server only issue?  What if you crate a
new server from scratch?

I will setup a fresh server to test this after work today and let you know

Alright.  I also did go through the code and verified adding the same ore
to the list in the config multuple times should increase the chances for a
given ore without and odd side effects

ok so I copied the server files exactly to a new directory nabt cp -R -> nabt-test
then I deleted the world directory and started it up. After building an orbital miner it's getting uninspected ores just fine.... But the primary server and the existing world there still isnt... this is super confusion now lol

That is really odd... maybe try running a diff on the configs on the new
server and configs on the old one.  If you don't get around to diffing the
startup logs I'll likely get to it within 2 hours.

Actually, hold on, I think I was a bit hasty with my last reply.  You said
you're using a copy of config and mods folder as the server with the
problem?  Stranger still... I'll add some print statements and build a
version with some print statements that should, I hope, shed light in the
issue

yeah its an exact copy of everything the only thing I deleted was the world folder which recreated, even used the same world seed etc.

AdvancedRocketry-1.10.2-1.1.7ExtraLog.jar.zip
Run this build on the server with the problems and post the generated log file.  I have it print when the ores are registered to the orbital drill and what ores are registered.  I'm wondering if it's being registered at the wrong time for some reason or something

done:
https://www.dropbox.com/sh/gzn7kle5ope7egj/AACi0fmfXz6zeqJFo4dc3FwXa?dl=0

Thanks.  It looks like it's not registering too early at least; well after the loadComplete phase.  But oddly enough that ore is not showing up in the list.  Gonna dig a bit deeper.

AdvancedRocketry-1.10.2-1.1.7.jar.zip
Would it be possible to run this build and grab the logs for it.  I've modified it to print out when ores are registered.  This way we can verify when it was registred

Yeah I'll run this soon. Just to make sure this part is clear, that ore has no intrinsic oredict, I'm specifically adding the oredict via minetweaker so if its attempting to load before minetweaker adds the oredict it wouldnt exist yet.

Yeah, the theory is maybe minetweaker is somehow screwed up or loads it after the laser checks for ores.

ok fresh set of logs:
https://www.dropbox.com/sh/gzn7kle5ope7egj/AACi0fmfXz6zeqJFo4dc3FwXa?dl=0

I have a theory.  I think what's going on is that FTBUtilities is loading chunks before the world is finished loading.  Because the spacelaser registers what can be mined when the first instance of the machine is loaded and it's now being loaded before minetweaker finishes registering recipes that this is occuring

Could you maybe load/build the cache a little later and just prevent the laser from running until the cache is built? I'm trying to test with ftbu chunk loading off but unfortunately turning chunkloading off doesnt clear chunks for loading it just prevents new ones from being added. Additionally wouldnt any chunkloading solution that loads it before the cache also potentially result in this issue?
alternatively, anyway to allow directly adding the ore like atmrockhounding:ore:1 to the list instead of an oredict entry?

I don't think I'll be able to cache it any later than I already am due to the fact the last thing that can happen is the block being loaded into the world.  The Ideal solution I thnink would be to have minetweaker load at ServerStartingEvent rather than ServerStarted, but I'm not sure if there's a reason they didn't do that and I don't maintain that mod, nor would I consider this situation their fault really.
I definatly could allow for parsing of blocks/items directly since I now have the infrastructure to do that and is the solution I'm gonna go with.

ok adding the ore directly will work perfect, I'm pretty sure MT loads when it does because loading early caused timeouts on connecting to servers due to JEI reloads, was a huge issue a month or two ago until it got moved.

I've made the change so now you can specify not only by name/meta but for size as well.
Ores can now be specified by just the oreDictName: or by ::: where size is optional.  So for example "minecraft:coal:1:32" is 32 charcoal, "oreTin:32" is 32 tin ores, and "oreCopper" is 5 copper ore (5 was the hardcoded size before)

do you happen to have a jenkins or anything where we could grab a copy of a version with these changes?


[BUG] KNNQueryBuilder sets kand minScore together

Describe the bug
I am integrating KNNQueryBuilder into my application to ensure k-NN queries are compatible with other query builders within the OpenSearch Java client. In the DSL, I typically utilize the minScore parameter, without specifying k or maxDistance. However, within KNNQueryBuilder, despite not explicitly setting the k parameter, it defaults to "k": 0. This results in an error at the client level:
"caused_by": {
   "type": "illegal_argument_exception",
   "reason": "[knn] requires exactly one of k, distance or score to be set"
}

The query builder looks like this:
KNNQueryBuilder knnQueryBuilder = new KNNQueryBuilder("name_vector", searchQuery.getEmbedding()).minScore(0.001f).boost(1);

and I use the default search method
this.client.search(searchRequest, RequestOptions.DEFAULT);

To Reproduce

Set minScore but not k parameter
Search it

What is the expected behavior?
I'm expecting if `the parameter k does not given at all (or set to be null), do not include it.
What is your host/environment?

Version OpenSearch 2.14

What could be a solution?
If the parameter k is passed as null, then do not write in doWriteTo method.

@safakkbilici Thanks for reporting the issue. We were aware of it but didn't thought there is a user using the KNNQueryBuilder directly from their application.
An alternative would be adding K-NN support in OpenSearch Java client.

@vamshin to triage

Lets close - recommendation is to use OpenSearch java client


*: update default etcd version to 3.1.2

A raft issue is fixed in 3.1.2 . Should recommend users to use it.

lgtm


  "message": "Unexpected error value: { name: \"SyntaxError\", message: \"Unexpected 'I'\", at: 1, text: \"Invalid Query\" }",

Hi
I am merging two API's in a same host using GraphQL. I followed what I done last time with Normal Openspec same thing I followed Here as well. But I am facing an issue like "Unexpected error value: { name: "SyntaxError". I am not sure Anything I added Wrong?
I am adding Code as well Anyone can take a look and give your inupt.
Code:
**_sources:

name: ScoopsSearch
handler:
graphql:
endpoint: http://localhost:8080/profiles/graphql/ScoopsSearch
baseUrl: http://localhost:8080/
operationHeaders:
user: "{env.user}"
x-zisession: "{env.x-zisession}"
x-ziid: "{env.x-ziid}"
x-ziaccesstoken: "{env.x-ziaccesstoken}"

name: FacetsSearch
handler:
graphql:
endpoint: http://localhost:8080/profiles/graphql/facets
baseUrl: http://localhost:8080/
operationHeaders:
user: "{env.user}"
x-zisession: "{env.x-zisession}"
x-ziid: "{env.x-ziid}"
x-ziaccesstoken: "{env.x-ziaccesstoken}"_**

Could you share a reproduction? Thanks

Sure. Let me share the GraphQL Playground screenshot.

Code

Sorry I need a reproduction on CodeSandbox, StackBlitz or at least in a repo. Screenshots don't help.

Hi @ardatan , It's my private office Repo... So that graphql service is accessible for me.... Any other option from your end.... The only problem when I call two times in same yaml file.... May I am available for a call any time ..  just 5 mins


Data-check-string

Validating data received via the Mini App
How do I go about checking the data?
Data-check-string is a chain of all retrieved fields, alphabetized in the format key=, with the line feed character ('\n', 0x0A) used as a separator - for example, 'auth_date=<auth_date>\nquery_id=<query_id>\nuser='.
how can I get this?

I've been trying for hours and I haven't been able to verify this data.  You can access this variable via TelegramWebApp.instance.initData.raw. But this is not sorted fields (it is just what is provided by telegram in the web app. When backend receives it, backend should decode, sort and remove hash field. Then calculate hashes.

Am I using the wrong package or is it the old version in pub.dev?
https://pub.dev/packages/telegram_web_app

@bugrevealingbme Sorry forgot to update with my local changes. Please check telegram_web_app 0.1.2

return TelegramInitData(
      user: user,
      chatInstance: chatInstance,
      chatType: chatType,
      authDate: authDate,
      hash: hash,
      raw: data,
    );

use
raw: rawData,
otherwise it bugs my app

not sure what changed but the plugin is broken

I downgraded and the problem was solved. I think this plugin is very important, could you please test it on the release version before you release it?

Great thanks). We are using this plugin in our production software. But I will try to test every time

Thank you, I am waiting for the new version. Also, does the start_param value work in version 0.1.0? I couldn't get it for some reason, how exactly do I use it?

You can ask topics in @flutter_telegram telegram channel.
Could you provide reproducing steps when the new version is causing the issue? As I said, latest version in my production code is working fine.
Regarding start_param,  you can start mini app with param in t.me/{botUsername}/{miniAppName}?startapp={param}. For example: t.me/bozorbop_bot/bozorchi?startapp=m402. You can access the variable via WebAppInitData::{start_param}.
There is another field tgWebAppStartParam with same functionality. But there is a known issue with this.

I don't do anything extra in the new version. Any call from TelegramWebApp.instance.initData directly breaks it.
I call start_param from initDataUnsafe but I can't get any data, the link I used: https://t.me/haw_haw_bot/theapp?start=165387110

I don't do anything extra in the new version. Any call from TelegramWebApp.instance.initData directly breaks it.
I call start_param from initDataUnsafe but I can't get any data, the link I used: https://t.me/haw_haw_bot/theapp?start=165387110

TelegramWebApp.instance.initDataUnsafe?.start_param

maybe it is broken to start_param in version 0.1.0. But I can't use 0.1.2

Have you tried running the package as telegram_web_app: ^0.1.2 instead of local?

Okay will check it today evening

Thanks, I am waiting to publish my application. Thank you for the package

I am closing this as it is resloved


[5.5] Allow marking notifications as unread

I have a notification area i show notifications to the users.
Some of my users need to mark notifications as unread to return for them later.
I think this is a very common thing, as the vast majority of email services allow for email to be marked as unread, also notifications in most of the websites are allowed to be marked as unread.

Please add tests.


Metadata Tables Cleanup Proposal

This PR contains a proposal for updating and normalizing our metadata tables. The outcome of this ticket is going to be a ticket that will be for the actual work, once we have solicited feedback from the team.

@LaFreda1 Do you know where we stand on this?

@arnejduranovic Are we moving off of Metadata tables? Can we close this ticket?


Fix telegraf tokens

To fix this, we need to remove the automatic token creation for telegraf configs in the ui, and add the ability for the user to select a token to use. We should show the list of tokens that have the right permissions. If no token exists, we should allow the user to create a new one (one click).
We should also run a migration to remove all the labels that "connect" the tokens to the telegraf configs
Originally posted by @russorat in https://github.com/influxdata/influxdb/issues/16059#issuecomment-561775757

I think we did this


Why is the program getting worse?

I use this program the way you say. I cannot get any results when I use the following command with version 1.1.3
XSpear -u "http://testphp.vulnweb.com" -d "searchFor:yy"
but with version 1.1.3 the following command was running.
XSpear -u "http://testphp.vulnweb.com/search.php?test=query&cat=123&ppl=1fhhahwul" -p cat,test
I installed version 1.1.6 today. but none of them work anymore.
XSpear -u "http://testphp.vulnweb.com" -d "searchFor:yy"
XSpear -u "http://testphp.vulnweb.com/search.php?test=query&cat=123&ppl=1fhhahwul" -p cat,test
previously running command also no longer works. The programs is correcting with updates but this program is getting worse.

From 1.1.6, only the parameters that have been reflected have been changed to test by default, except blind xss and basic tests. (because too many requests occur and have a significant impact on scan speed)
So I don't think the results are there. Can you give me a details log?
e.g
xspear  -u "http://testphp.vulnweb.com/search.php?test=query&cat=123&ppl=1fhhahwul" -p cat,test
    )  (
 ( /(  )\ )
 )\())(()/(          (     )  (
((_)\  /(_))`  )    ))\ ( /(  )(
__((_)(_))  /(/(   /((_))(_))(()\
\ \/ // __|((_)_\ (_)) ((_)_  ((_)
 >  < \__ \| '_ \)/ -_)/ _` || '_|
/_/\_\|___/| .__/ \___|\__,_||_|    />
           |_|                   \ /<
{\\\\\\\\\\\\\BYHAHWUL\\\\\\\\\\\(0):::<======================-
                                 / \<
                                    \>       [ v1.1.6 ]
[*] analysis request..
[-] [09:46:49] [200/OK] 'cat' not reflected <script>alert(45)</script>
[-] [09:46:49] [200/OK] 'test' not reflected rEfe6
[-] [09:46:49] [200/OK] 'cat' not reflected rEfe6
[-] [09:46:49] [200/OK] 'STATIC' not reflected
[-] [09:46:49] [200/OK] 'test' not reflected <script>alert(45)</script>
[*] creating a test query [for reflected 0 param + blind xss ]
[*] test query generation is complete. [0 query]
[*] starting XSS Scanning. [10 threads]
[*] finish scan. the report is being generated..
+----+-------+-----------------+--------+-------+------------------+---------------------------+
|                                      [ XSpear report ]                                       |
|        http://testphp.vulnweb.com/search.php?test=query&cat=123&ppl=1fhhah... (snip)         |
|            2019-08-19 09:46:48 +0900 ~ 2019-08-19 09:46:49 +0900 Found 5 issues.             |
+----+-------+-----------------+--------+-------+------------------+---------------------------+
| NO | TYPE  | ISSUE           | METHOD | PARAM | PAYLOAD          | DESCRIPTION               |
+----+-------+-----------------+--------+-------+------------------+---------------------------+
| 0  | INFO  | STATIC ANALYSIS | GET    | -     | <original query> | Found Server: nginx/1.4.1 |
| 1  | INFO  | STATIC ANALYSIS | GET    | -     | <original query> | Not set HSTS              |
| 2  | INFO  | STATIC ANALYSIS | GET    | -     | <original query> | Content-Type: text/html   |
| 3  | LOW   | STATIC ANALYSIS | GET    | -     | <original query> | Not Set X-Frame-Options   |
| 4  | MIDUM | STATIC ANALYSIS | GET    | -     | <original query> | Not Set CSP               |
+----+-------+-----------------+--------+-------+------------------+---------------------------+
< Available Objects >
Not found

< Raw Query >

p.s.
The format of the data in the -d option is abcd=1234 and JSON POST is not yet supported.
I'm still thinking about "default reflected" and "default all." If you have any good ideas, please share them.


Fix DateFormatter behaviour for nil locale and timeZone.

The locale and timeZone properties of DateFormatter seems to be "null
resetteable" in macOS, but the Swift Foundation behaviour differs.
Before this patch, setting locale to nil would segfault the program the
next time the formatter is used, while timeZone will incorrectly return
nil. After this patch a nil locale will simply reset to the current
locale and a nil timeZone will reset to the system one.
The tests check that the initial defaults are also correctly set.

@swift-ci please test


Allowing users to integrate external checks via blank-imports

Original proposal
In #3055 @naveensrinivasan proposed using the plugin functionality (https://pkg.go.dev/plugin) to handle:

It would be nice for users to be able to run their own custom checks in addition to those available in Scorecard.

While plugin exists, it comes with many constraints.

Runtime crashes are likely to occur unless all parts of the program (the application and all its plugins) are compiled using exactly the same version of the toolchain, the same build tags, and the same values of certain flags and environment variables.
Similar crashing problems are likely to arise unless all common dependencies of the application and its plugins are built from exactly the same source code.
Together, these restrictions mean that, in practice, the application and its plugins must all be built together by a single person or component of a system. In that case, it may be simpler for that person or component to generate Go source files that blank-import the desired set of plugins and then compile a static executable in the usual way.

blank-import
Instead, I want to focus on the blank-import approach, as that's what scorecard does today for its own checks:
https://github.com/ossf/scorecard/blob/bfd535755379bbe13d156b262fc9dd8ec33e23a4/checks/binary_artifact.go#L24-L36
If we make the necessary changes to allow external modules to do the same, custom checks becomes a possibility:
Imagine some develops a custom, external check in their repo: github.com/foo/mycustomcheck
package mycustomcheck // github.com/foo/mycustomcheck

import (
	"github.com/ossf/scorecard/v4/checker"
	"github.com/ossf/scorecard/v4/checks"
)

const name = "My-Custom-Check"

func init() {
	checks.RegisterCheck(name, MyCustomCheck, nil)
}

func MyCustomCheck(r *checker.CheckRequest) checker.CheckResult {
	return checker.CreateMaxScoreResult(name, "my check found nothing wrong!")
}

If a consumer uses a small wrapper around scorecard's main function, the check could be integrated just by adding blank-imports, without modifying the scorecard code themselves:
package main

import (
    "log"

    "github.com/ossf/scorecard/v4/cmd"
    "github.com/ossf/scorecard/v4/options"

    _ "github.com/foo/mycustomcheck"
    _ "github.com/bar/someothercheck"
)

func main() {
    opts := options.New()
    if err := cmd.New(opts).Execute(); err != nil {
        log.Fatalf("error during command execution: %v", err)
    }
}

Changes required

Export RegisterCheck
Figure out an alternative to checkDocs.GetCheck(check.Name) which will work with external checks

need to get things like the risk to determine score weight, documentation links, etc.

With some hacky local modifications (see Changes required), the above code works at a PoC level:
go run main.go --repo ossf/scorecard --checks My-Custom-Check --format json | jq
{
  "date": "2023-05-31T12:30:54-07:00",
  "repo": {
    "name": "github.com/ossf/scorecard",
    "commit": "bfd535755379bbe13d156b262fc9dd8ec33e23a4"
  },
  "scorecard": {
    "version": "",
    "commit": "unknown"
  },
  "score": 10,
  "checks": [
    {
      "details": null,
      "score": 10,
      "reason": "my check found nothing wrong!",
      "name": "My-Custom-Check",
      "documentation": {
        "url": "",
        "short": ""
      }
    }
  ],
  "metadata": null
}

Amazing! Thanks @spencerschrock

Export RegisterCheck

Instead of exporting registerCheck, it may make more sense to have a RegisterExternalCheck to handle any differences we want to enforce between standard and external checks.


Cannot connect to application

I am deploying my application to Azure using the Demeteorizer tool but when I try and connect to Kadira, there is no logging.

Also, this is what my settings.json looks like. Keep in mind, this configuration works locally, maybe I am missing something?
{
  "database": {
     "server"   : "*********.database.windows.net",
     "database" : "********",
     "user"     : "****",
     "password" : "****",
     "options"  : {
        "encrypt" : true,
        "appName" : "Test App"
      }
  },
  "kadira": {
     "appId": "********",
     "appSecret": "*******-*****-*****-****-*******"
  }
}

Thanks in advance!

Kadira shut down yesterday May 15 11.59 PM Pacific Time.
From an email Arunoda sent out
Hi there,
As we discussed earlier, we were supposed to shutdown by April. But with the Meteor's acquisition of Kadira assets we both have decided to keep it maintain for until May 15th.
So, we'll officially shutdown kadira.io service by May 15 11.59 PM Pacific Time.
We did not migrate accounts to Meteor's APM (a version of Kadira) and all the accounts (including payments) will be deleted. You can use use Meteor APM or open source version of Kadira if you need to continue using Kadira.
For more information, just reply to this email.


Scripts are not loaded synchronously within the same Helmet component

I would like to inject 2 script tags like this:
jQuery is just an example
      <Helmet>
        <script
          src="https://code.jquery.com/jquery-3.3.1.min.js"
          integrity="sha256-FgpCb/KJQlLNfOu91ta32o/NMZxltwRo8QtmkMRdAu8="
          crossorigin="anonymous"
        ></script>
        <script>
          {`
            console.log('Test', typeof $);
          `}
        </script>
      </Helmet>

After the first script is loaded I want to execute the second inline script. Normally, when you put 2 scripts into HTML without any async or defer they will load synchronously.
Currently both scripts will be executed asynchronously. See reproduction here - https://codesandbox.io/s/l9qmrwxqzq

Unfortunately this is a known issue. We need to do a better job at communicating this bug to the everyone.
Note to self add a FAQ to the readme related to this bug.

@tmbtech definitely :) I was looking for it through the issues and did not found it

In the meantime, internally we do something similar to this , maybe it'll help.

class LoadExternalScript extends React.Component {
    static defaultProps = {
        onLoad: () => {},
        onError: () => {},
    };

    render() {
        const {props} = this;
        if (!canUseDOM) {
            props.onError("DOM not found");
            return false;
        }

        if (!document.getElementById(props.id)) {
            const script = document.createElement("script");
            script.src = props.src;
            script.id = props.id;
            script.onload = props.onLoad;
            script.onerror = props.onError;

            if (document.body) {
                document.body.appendChild(script);
            }
        } else {
            props.onError("script already loaded");
        }

        return false;
    }
}

// some other file

class Foobar extends React.Component {
  state = {
      status: "pending"
  }
  render() {
     return (
         <LoadExternalScript
              src="example.com"
              id="example"
              onLoad={() => this.setState({status: "scriptLoaded"})}
         />
     )
  }
}

<Foobar /> can then have additional checks on if needs to load a script again and what components it needs to render after the global js script has loaded.
or depending on your case, you might just want to load the script in the html template? idk just throwing out ideas.
goodluck.

Any workaround?

Still having the same issue here.
Is there any way to fix it?

Still have this issue. I have a bunch of  in , but they seems to not execute in the order they are declared.

It seems that the script elements created by helmet have the async attribute and AFAIK there is no way how to disable it (e.g. by passing async={false} in JSX - this just sets the value of async to "false" but for unsetting it, the attribute would have to be removed).

@zub2 setting async={false} as you suggested just worked for me! I'm using Helmet 6.1.0 within Gatsby 2.23.3 with TypeScript

@zub2 setting async={false} as you suggested just worked for me! I'm using Helmet 6.1.0 within Gatsby 2.23.3 with TypeScript

did you pass async={false} to every script or to your Helmet HOC ?

This bug seems pretty old. Any update on this?
https://stackoverflow.com/questions/65319422/append-multiple-script-tags-to-head-that-execute-synchronously-in-react/

Super annoying that this is still a thing and that it's not documented in any obvious way. Took me way too long to figure out what was going on with my scripts. The readme says "Helmet takes plain HTML tags and outputs plain HTML tags. It's dead simple, and React beginner friendly." But plain HTML script tags load synchronously, and Helmet changes this behavior on the sly; that's not simple or beginner friendly.

Super annoying

Just found this. The onLoad attribute is also not set properly.
const handleScriptLoaded = () => { console.log("**** LOADED"); };

<Helmet>
   <script src="https://domain.com/path/to/script.js" onLoad={handleScriptLoaded} type="text/javascript" />
</Helmet>

Creates the element:
<script src="http://app.thermoform.localhost/auth/js/api.js" onload="() => {
    console.log("*** LOADED");
  }" type="text/javascript" data-rh="true"></script>

This is completely unusable.

just another workaround
i'm giving my script setTimeout with 50ms, and it's worked fine and still acceptable on my side

There was no workaround so this is what I used:
function getScript(script: string) {
    return new Promise((resolve, reject) => {
        const $script = document.createElement("script");
        $script.onload = resolve;
        $script.onerror = reject;
        $script.src = script;
        document.head.appendChild($script);
    });
}

function useScripts(user_scripts: string[]) {
    useLayoutEffect(() => {
        const scripts = [...user_scripts];
        (function loop() {
            if (scripts.length) {
                const script = scripts.shift();
                getScript(script).then(loop);
            }
        })();
    }, []);
}

You can add a listener in the Helmet and use react states to check if the first script is loaded before executing the second . To do so use the onChangeClientState Helmet prop


Allow enums to have constants as values

Hi I think I found how to fix the issue, it's working for me.
Issue 24
Regards,
Leon

I also changed the exceptions messages to help me debug.

 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.Leon Moctezuma seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[folly] update to v2<IP_ADDRESS>

Fix #17441
Update port folly to v2<IP_ADDRESS>
All feature are tested successfully in the following triplets:

x64-windows
x64-windows-static

Included in #17335


Fix Markdown syntax in README to RST

@dhermes sorry about that, this time I looked and it looks right.

Indeed, looks good: https://github.com/waprin/google-cloud-python/blob/fix_rst/README.rst
I am happy to merge, though should #python be in code font? (Your call)

done and done


Improvement for JS ecosystem

Currently kolmafia can't be imported outside of a KoLmafia environment because ESM knows that the named exports aren't actually there. I would like to do this so I can share code between the web and a mafia script, knowing that I will not actually execute mafia-specific code. This fixes that

Without usage statistics I would imagine most KoLmafia users execute JS scripts at least once day, which this could affect if it were wrong.

Small mistake in the github action, I will push to main to fix.

Without usage statistics I would imagine most KoLmafia users execute JS scripts at least once day, which this could affect if it were wrong.

I would be astonished if this were true.
Perhaps you mean "most KoLmafia users who are on Discord" - which I am sure is a minority of all KoLmafia users.

No I didn't mean that at all. I really just mean garbage-collector which is
frequently mentioned in in-game chat, moreso than any other script I see
referenced by name. But again, my anecdote and I could easily be wrong.
Just a comment in passing really.
On Tue, 9 Apr 2024 at 17:12, Veracity0 @.***> wrote:

Without usage statistics I would imagine most KoLmafia users execute JS
scripts at least once day, which this could affect if it were wrong.
I would be astonished if this were true.
Perhaps you mean "most KoLmafia users who are on Discord" - which I am
sure is a minority of all KoLmafia users.
—
Reply to this email directly, view it on GitHub
https://github.com/kolmafia/kolmafia/pull/2270#issuecomment-2045579900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABL4LVNFRBGCRYWAT2P5UTY4QHP3AVCNFSM6AAAAABF4RMQ2SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBVGU3TSOJQGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Without usage statistics I expect most KoLmafia users don't execute any scripts at all.


Japanese Language file

My personal Japanese translation of the
English language file.

Wow ...perfect ! thanks a lot for your commit !!!!

No problem. Glad to help. I might take a crack at some issues if I feel up to it.


Plugin-specific colors are overridden

First of all, thanks for a great colorscheme! 🙂
I have been having some issues with the signs used by vim-gitgutter. For some colorschemes the background is highlighted although I want only the foreground to be highlighted. I tried using let g:gitgutter_set_sign_backgrounds = 1 in my config without any luck.
I then noticed that for all the colorschemes with the "incorrect" background color, the plugin groups, e.g. GitGutterAdd, were last set by hardhacker in hardhacker/after/plugin/hardhacker.vim. This file calls s:AfterHighlight when sourced and links some plugin highlight groups using :hi! which forcibly overrides any existing links. For some reason, this happens for some colorschemes but not for others but the chosen colors might also just coincide with the sign column background color.
If I then run colo <current_colorscheme> for a colorscheme with overridden colors or switch to another colorscheme with the problem, then the background color disappears from the signs.
I've confirmed that the function runs every time I open a file as it should run after the main hardhacker plugin/colorscheme has been loaded. I also tried to disable all other plugins  except colorschemes and vim-gitgutter but the issue persists. Uninstalling hardhacker fixes the issue.
I would imagine that the s:AfterHighlight should only be run after something like :colo hardhacker, include a check that the current colorscheme is hardhacker, or use an autocommand?
Here are two screenshots showing the issue:

If it's relevant:

neovim v0.9.0
Mac osx Big Sur 11.7.7
Installing hardhacker with Plug 'hardhackerlabs/theme-vim', { 'as': 'hardhacker' }

thanks for your feedback, I will optimize the colorscheme and fix the issue about s:AfterHighlight.

I have optimized the colorscheme and fixed the issue about AfterHighlight
Thanks agin, You can update and try it.

I assume you've already confirmed this since the issue was reopened but I tried updating but the issue still persists unfortunately.